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

External perimeters not given adequate detail #4640

Closed
VanessaE opened this issue Dec 9, 2018 · 3 comments
Closed

External perimeters not given adequate detail #4640

VanessaE opened this issue Dec 9, 2018 · 3 comments

Comments

@VanessaE
Copy link
Collaborator

VanessaE commented Dec 9, 2018

Describe the bug

This is Slic3r commit 21eb603 plus @supermerill's thin wall improvements PR (I'm at commit ca9f3f7 of that).

Slic3r's discarding small features in the model instead of trying to recreate them. Consider the nameplate on the back of a Benchy:

screenshot_2018-12-09_01-16-45-2

0.4 mm nozzle, 0.4 mm line width, 0.08 mm layer height (config: config-20181209.ini.zip ).

Compare to this photo I found on Hackaday. While the image and print quality are mediocre, it clearly shows all of the lettering's individual strokes. This was apparently also printed with a 0.4 mm nozzle, sliced with Cura (not sure about the layer height, but it looks to be about 0.1 mm).

benchyback

(rotated, cropped, resized, and I painted-out a blob that was in the middle of the plate, for easier visual comparison; source: https://hackaday.com/2016/08/30/review-monoprice-maker-ultimate-3d-printer/ ).

@supermerill
Copy link
Collaborator

supermerill commented Dec 10, 2018

for the '#' : the thin area between the two '/' is too small for a 0.4 nozzle to do a U.

But there should be an extrusion between 3,D,b

If you disable the thin_wall option, you can have it.

The "culprit" is done at perimetergenerator.cpp l85 vs l97. I didn't change it (i just added a little post-process for thin wall to relax the algo a bit if it creates too much ares).

What i understand : If no thin wall, it lay every perimeter the best way possible. If thin wall, it exclude areas that can be too thin for 2 perimeters (you can se it, when 2 perimeter are almost one over another -> overextrusion), and pray for the thin walls to extrude these thin area. The problem is: sometimes, these areas are too tiny.

I will try something.

supermerill referenced this issue in supermerill/SuperSlicer Dec 10, 2018
supermerill referenced this issue in supermerill/SuperSlicer Dec 10, 2018
supermerill referenced this issue in supermerill/SuperSlicer Dec 10, 2018
@VanessaE
Copy link
Collaborator Author

This is one of those situations where Cura, for example, would probably do those overlapped perimeters I mentioned.

@VanessaE
Copy link
Collaborator Author

Looking much better with the latest updates on the thin walls PR (commit 44e7ec8):

screenshot_2018-12-21_03-11-18

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