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

Inner/outer/inner with 2 walls starts with outer instead of inner wall #7997

Open
3 tasks done
tlhintoq opened this issue Jan 10, 2025 · 7 comments
Open
3 tasks done
Labels
bug Something isn't working

Comments

@tlhintoq
Copy link

Is there an existing issue for this problem?

  • I have searched the existing issues

OrcaSlicer Version

2.3-dev

Operating System (OS)

Windows

OS Version

Win10proX64

Additional system information

StealthDeck_DeviceManager

Printer

Any

How to reproduce

  1. New job
  2. Add a cube
  3. 2 walls
  4. inner/outer/inner order
  5. slice

Actual results

Outer wall prints before inner

Expected results

Inner/outer/inner
if there's no 3rd wall then still should be
Inner/outer/...

Project file & Debug log uploads

2025-01-10_19-22-50

Checklist of files to include

  • Log file
  • Project file

Anything else?

No response

@tlhintoq tlhintoq added the bug Something isn't working label Jan 10, 2025
@discip
Copy link
Contributor

discip commented Jan 10, 2025

@tlhintoq
If you are using the latest dev build this might be related to #7951 and if so will be fixed with the merge of #7959. 😊

@tlhintoq
Copy link
Author

@tlhintoq If you are using the latest dev build this might be related to #7951 and if so will be fixed with the merge of #7959. 😊

Except I don't use the Arachne engine (due to the problems in it like that one).
As seen in the screen capture this is with Classic Wall Generator

@discip
Copy link
Contributor

discip commented Jan 10, 2025

Ups. 😅
Sorry, I missed that crucial detail. 🤦‍♂️

@buzzhuzz
Copy link
Contributor

Sorry my ignorance, but how inner/outer/inner supposed to work with only 2 perimeters?

I've just looked into the feature tooltip and it is clearly says "this feature required minimum of 3 walls to be effictive..."

@MxBrnr
Copy link

MxBrnr commented Jan 11, 2025

I would argue this is working exactly as intended. The whole purpose of inner/outer/inner wall order is so that the outer wall is printed before the immediately adjacent inner wall. In the case of a 2 wall print, you are simply eliminating the first inner wall, effectively changing this setting to outer/inner. As previously mentioned and detailed in the tooltip, inner/outer/inner only works with 3 or more walls.

@tlhintoq
Copy link
Author

tlhintoq commented Jan 12, 2025

to be effictive...

'Effective' is not the same as work in sequence as described. It may not be as effective with only 2 walls but it should still follow the same sequence.

@MxBrnr That's one way to look at it.

The other is... Its called inner/outer/inner.
That's the order it should print in.
1 - inner
2 - outer
3 - inner

The whole purpose of inner/outer/inner wall order is so that the outer wall is printed before the immediately adjacent inner wall.
One interpretation.

The other is:
"The inner wall should print before the outer so there is something for the outer wall to stick to on slopes/overhangs."

Here's the common workflow...
I'm debating between 2 walls and three... Wall order of (1)inner/(2)outer/(3)inner
I slice it at 3... And I get an inner wall before the outer wall. "Ok, I like that. But I don't need 3 for this object."
So I reduce it from 3 walls to 2. That should be all that needs to be done, and I should still get the (1)inner wall before the (2)outer wall because there is no (3)inner l to make

@henrivdr
Copy link
Contributor

I agree 100% with @tlhintoq

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

No branches or pull requests

5 participants