slic3r / Slic3r

Open Source toolpath generator for 3D printers
https://slic3r.org/
GNU Affero General Public License v3.0
3.33k stars 1.29k forks source link

When "combine infill" > 1, solid top layers should use combined infill height rather than perimeter height #2872

Open triffid opened 9 years ago

triffid commented 9 years ago

Yesterday I did a print that has a very shallow dome. I set "combine infill" to 5 layers and layer height to 0.05, which should give infill height of 0.25. This is the perfect usage case for this wonderful feature.

Unfortunately, I didn't also change "solid top layers" to 20-25, and slic3r seemed to try to put 0.05mm fill on top of 20% honeycomb fill which just plain does not work - see #240 and images of terrible top-layer acne

To me it would make more sense for slic3r to use infill height for solid top fill (ie 0.25mm) in this situation, rather than 2-20 layers of 0.05 solid fill. The retreating perimeter line should cover the coarse infill neatly in this case.

Perhaps, when slic3r detects that combined fill is intersecting other combined fill with a different offset (eg infill at 2.2 and infill at 2.25 due to domed top) it could lower the higher one, and put some thin (0.05) fill on top to bring it up to height?

This might help avoid having little bits of infill scattered all over the place.

Would detecting domes in the model by searching for large connected groups of facets with not-quite-vertical normals help?

config/STL/Gcode/ 310212ed30b593c86ef502849e8c9c93050a3f49

alranel commented 9 years ago

We used to do that, but another report pushed us to limit the effects of "combine infill" to the internal infill. So we're in a "perpetual regression" loop. :-)

We can make that configurable. Or smarter, somehow.

harriv commented 9 years ago

What if top layers were millimeters instead of layer count?

triffid commented 9 years ago

I would like quite a number of settings to accept millimeters, eg perimeter width, top/bottom infill as you say, maybe convert "combine infill" to "infill layer height (mm)"

I think having top/bottom fill as millimeters rather than layers would help, but we'd (apparently) still need a setting for whether to use infill layer height or perimeter layer height for filling this volume.

kefir- commented 9 years ago

I'm personally not a fan of specifying these things in mm, I prefer layer count. If course, if slic3r accepts both, that's fine. But slic3r could offer to recalc perimeter settings and top/bottom layer counts to keep the same thickness when layer height or other settings are changed that affect them.

mrvn commented 8 years ago

@kefir- I think neither thickness in mm not layer count is the right option, you need both (and more?).

So I would suggest 3 settings:

That way the same settings could be used for any number of layer heights.

MortarArt commented 6 years ago

I feel like with the adaptive quality settings in place, the request for measurement based top and bottom layer heights is even more important.