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

Remove "auto" wall direction option #6193

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

vovodroid
Copy link
Contributor

@vovodroid vovodroid commented Jul 20, 2024

Refactoring of Allow specify wall directions #4156.

Simplify wall direction option. It assists with further wall related development. For example I separated between internal and overhang walls reversing, and added option to print overhangs after internal to improve adhesion while keeping external wall first for outer-inner and inner-outer-inner modes.

  1. Auto mode was removed. Just turn reversing on/off for both direction.
  2. Spiral vase could be printed in any direction.

I hope current direction logic was kept. Let me know if something is missed.

@CristianGTR97
Copy link

very clever

@vovodroid vovodroid force-pushed the wall-dir-refactor-pr branch from 966b99e to 40dd3ac Compare September 9, 2024 07:48
@discip
Copy link
Contributor

discip commented Sep 9, 2024

@vovodroid
Is the unfortunate behavior I mentioned here #6236 (comment) also addressed in this PR?

@vovodroid
Copy link
Contributor Author

No, this PR is only base for other changes addressed in my fork. Probably these changes fix your issue.

@vovodroid vovodroid changed the title Refactor wall direction Remove "auto" wall direction option Oct 9, 2024
@vovodroid vovodroid force-pushed the wall-dir-refactor-pr branch from 0f2636d to c2b304b Compare October 12, 2024 10:42
@vovodroid vovodroid force-pushed the wall-dir-refactor-pr branch from c2b304b to 14eea1d Compare January 10, 2025 20:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants