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

Fixes scars on thing walls when "avoid crossing wall" is turned on #2659

Merged
merged 2 commits into from
Nov 9, 2023

Conversation

Noisyfox
Copy link
Collaborator

@Noisyfox Noisyfox commented Nov 8, 2023

This fixes #2621

Before:
25aa3d6cc7a83cdf505a55316bb13f65

After:
512bd63453cce8b1775bf52abc9afedb

These fixes are cherry-picked from PrusaSlicer.

…when the avoid crossing perimeters was enabled.

(cherry picked from commit prusa3d/PrusaSlicer@6c51e51)
…e scars on thin objects (SoftFever#7699).

Previously, the minimum contour width was chosen too conservative and, on some thin objects, only allowed minimal (or non) offset. This could result in travels being planned along the outer perimeter.

Now, the minimum contour width is chosen much smaller at the start and tested if the variable offset wasn't failed (the outer contour broke up into more parts, more or fewer holes, etc.).
If any problem is detected, the variable offset is recalculated with a larger minimum contour width.

(cherry picked from commit prusa3d/PrusaSlicer@dc00f0b)
Copy link
Owner

@SoftFever SoftFever left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks

@SoftFever SoftFever merged commit f16feff into SoftFever:main Nov 9, 2023
4 checks passed
@Z18Aplha
Copy link

Z18Aplha commented Nov 9, 2023

Great Work! Thank you very much!

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.

Avoid crossing walls - rough walls
4 participants