slic3r / Slic3r

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

The randomize starting point option is ignored for single walls #924

Open llluis opened 11 years ago

llluis commented 11 years ago

Hi, I just sliced the 40x10.stl (from Slic3r github) with only 1 wall, no infill and "Randomize Starting points" OFF, however, Slic3r created two start points alternating each layer.

alranel commented 11 years ago

Okay, that happens with thin walls and should be fixed.

We always split them at first point and store them as ExtrusionPath objects rather than ExtrusionLoop, and this is why the randomization isn't applied. I think we can get rid of the whole ExtrusionPath::Collection object after we merge medial-MGD and only store individual ExtrusionLoop objects.

alranel commented 10 years ago

I fixed this for normal perimeters, but the issue is still present with thin walls because internally they're not kept as loops.

callimero commented 9 years ago

I just stumbled on this issue. Is there a solution in sight? My workaround was to thicken my walls a bit so I dont need to use "Detect thin walls". But it would be nice to not do this. Maybe for now a hint in the GUI that thin walls can make Random Seam Position not working?

EDIT: Hrm... Not a good workaround, getting overextrusion... Terrible quality. Would be really nice to have Random Seam together with Thin Walls