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

Thin Wall Problem #328

Closed
JoduWei opened this issue Jun 30, 2020 · 5 comments
Closed

Thin Wall Problem #328

JoduWei opened this issue Jun 30, 2020 · 5 comments
Labels
duplicate This issue or pull request already exists working as intended unless you prove me wrong.

Comments

@JoduWei
Copy link

JoduWei commented Jun 30, 2020

Version

Current revision + 2.2.52.0_win64_20200623

Operating system type + version

Windows 10

Behavior

See Screenshots

thinwall

a_drive_frame_lower.3mf.zip

@supermerill supermerill added working as intended unless you prove me wrong. duplicate This issue or pull request already exists labels Jun 30, 2020
@supermerill
Copy link
Owner

#150

@n8bot
Copy link

n8bot commented Jun 30, 2020

The slicer is avoiding the overlap of the two external perimeters from the hole and the outside of the part, where they become so close as to require this. As @supermerill mentions in the issue linked, you need to enable the option to allow overlap of perimeters.

On the topic of the overlap of perimeters, Slic3r default behaviour is indeed to overlap all paths slightly.

From Slic34 Manual —­ Flow Math:

Path spacing is thus:

spacing = extrusion_width - layer_height * (1 - PI/4)

Is this behaviour still present in SuperSlicer? I'm not actually familiar with coding enough to understand everything and determine this answer myself. Sorry if this is a stupid question.

@supermerill
Copy link
Owner

supermerill commented Jun 30, 2020

yes it's the same,
but since 2.2.52, you can change that for perimeters with the perimeter overlap (% of the right part of the formula).

@n8bot
Copy link

n8bot commented Jun 30, 2020

Oh, nice!

I guess this is what is meant by this note on the release page:

  • perimeter overlap #4 perimeter overlaps : total control over perimeter width vs spacing. These settings may allow you to retain the same perimeter quality when you under-extrude but without under-extruding. Also may help to hide infill pattern passing through the perimeter. These needs some experimentation to see if they are useful.
    • external_perimeter_overlap to control half of the overlap between the external perimeter and the inner one: 0% is no overlap, 100% is full overlap (current & default value)
    • add perimeter_overlap, same between every perimeter (and the other half with the external one)
    • add perimeter_bonding, that shrink the external-internal spacing, without growing the infill polygons.

Would you be able to point me to the area in the source code that this is done? Just the general direction if it's complex. I'm just curious, I don't have any intention on modifying anything.

@supermerill
Copy link
Owner

supermerill commented Jun 30, 2020

perimetergenerator.cpp (in the begining) for the use of it.
flow.cpp for the multiplication.
you can also search for the commit that reference the issue 4

edit: you can modify if you want, it's open source!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate This issue or pull request already exists working as intended unless you prove me wrong.
Projects
None yet
Development

No branches or pull requests

3 participants