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

[FR] Import and finish-up "Thin wall improvements (v3)" from upstream Slic3r #4422

Open
VanessaE opened this issue Jun 22, 2020 · 3 comments

Comments

@VanessaE
Copy link

VanessaE commented Jun 22, 2020

Version

git commit 9189758

Operating system type + version

Debian buster

3D printer brand / version + firmware version (if known)

Custom Hypercube running Marlin bugfix-2.0.x (at commit 8000182b).

Behavior

For the longest time now, thin wall behavior on Slic3r and Prusaslicer has been of sub-standard quality - bends at the ends of lines, over-extrusion, sometimes just not being drawn, and a few other things.

See slic3r/Slic3r#4534 followed slic3r/Slic3r#4790 for more details on the existing code's behavior, and discussion on how fixes were being implemented into Slic3r (it worked great, but has not been finished/merged).

@supermerill would you consider porting your Slic3r (or SuperSlicer) implementation to Prusaslicer?

Project File (.3MF) where problem occurs

(N/A)

@supermerill
Copy link
Contributor

Sure, it's not that hard,
It's basically the changes from the files in the pr, but from my current fork master branch.

Their is some things why i didn't did it yet:

  • I can still discover some sub-optimal behavior, so it evolve a bit over time.
  • I don't want to bother and waste prusa time a third time on this issue, until my "solution" is almost perfect, as the previous improvements weren't good enough. So the goal was the pr merge into the upstream before submitting again.
  • I didn't merged the reforged PerimeterGenerator from prusa. Have to do that before creating the pr.

@VanessaE
Copy link
Author

@supermerill any movement on this?

@supermerill
Copy link
Contributor

I have to finish the upstream one first.
I also added a little code to merge the thin walls into the perimete loop, to avoid oozing and retractions. Need some more time to perfect it.

Didn't had the time yet to merge prusa's perimetergenerator changes. Maybe on the next big merge.

If someone else want to submit a pr here, you can do it. It's basically the new class medialaxis.cpp and "some" changes in perimetergenerator.

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

No branches or pull requests

2 participants