Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Incorrect wall order when inner-outer-inner/infill and 2 walls selected #1366

Closed
alexrmay91 opened this issue Jun 16, 2023 · 8 comments
Closed
Labels
bug Something isn't working stale

Comments

@alexrmay91
Copy link

Describe the bug
I set Order of inner wall/outer wall/infill to 'inner-outer-inner/infill' for almost all prints. This functions correctly unless Wall loops is set to 2. When both of these are set, inside-loops (i.e. holes) print outer-inner while outside loops (i.e. outside of the object) print outer-inner. See the example below where the outside wall is printing first on the inner loop, opposite on the outer loop.

3mf File for This Bug
All models with any type of hole inside the part.

To Reproduce
Use the settings listed above. Select 3 walls or inner/outer/infill for normal behavior.

Expected behavior
When set to inner-outer-inner/infill and 2 walls, it should print with the inner wall first, especially on inside holes. This behavior leads to print failures, especially on models with countersunk holes (or other internal overhang). The walls tend to not stick to the previous layer and end up crossing the middle of the hole.

Screenshots

image

image

Printer model
Voron Trident

Desktop (please complete the following information):

  • Windows
  • V1.6.3
@alexrmay91 alexrmay91 added the bug Something isn't working label Jun 16, 2023
@igiannakas
Copy link
Contributor

igiannakas commented Jun 17, 2023

This is not a bug, it is supposed to work this way. The inner outer inner by default needs 3 perimeters to work - the first inner is the third wall, then it prints the outer perimeter then the inner perimeter next to the wall. So if you don’t have three perimeters the first “inner” is not there to print.

This is meant to work like the outer perimeter first mode but rather print an internal wall first away from the outer wall in order to improve the artefacts on the z seam that may occur from using outer perimeter first.

@alexrmay91
Copy link
Author

I understand how it works. I think my original posts may not have been clear?

I leave the wall order to inner-outer-inner by default, but sometimes print with 2 walls. When that happens, the walls on inside hole features print in a different order than walls on outside features. Look at the photos/toolhead. Picture 1, inside feature, orange is being layed down first. Picture 2, outside feature, yellow before orange. These pictures are using the same settings, no re-slicing.

@igiannakas
Copy link
Contributor

igiannakas commented Jun 17, 2023

I’ve fixed this issue in my PR #1352. In this Pr outside walls are favoured over inside.

If you can upload the stl I’ll check if it happens with the updated version I’ve submitted.

@alexrmay91
Copy link
Author

The model in the screenshots is from here on printables, the small version.

@igiannakas
Copy link
Contributor

The model in the screenshots is from here on printables, the small version.

With the PR #1352 it is printing consistently with outer wall first when only 2 walls are selected as it falls back to the outer perimeter first logic.

Screenshot 2023-06-17 at 19 18 27
Screenshot 2023-06-17 at 19 18 57

When three walls are selected it prints the innermost perimeter correctly, then the outermost.
Screenshot 2023-06-17 at 19 20 33

@alexrmay91
Copy link
Author

I see. Sounds like I'll just have to remember to turn it back to inner-outer when I use two walls. Thanks for following up.

Copy link

github-actions bot commented Nov 6, 2023

GitHub bot: this issue is stale because it has been open for 90 days with no activity.

@github-actions github-actions bot added the stale label Nov 6, 2023
Copy link

GitHub bot: This issue was closed because it has been inactive for 7 days since being marked as stale.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working stale
Projects
None yet
Development

No branches or pull requests

2 participants