Align Seams should consider the previous, rather than the next path. about compas_slicer HOT 4 CLOSED

I think I see what you mean but I'm not sure what this would change? This would only have an effect on the first path of a layer right? Because currently the seams_align method never changes the first path (path0), instead it aligns the seam of path1 such that it is closest to the seam of path0, then it aligns the seam of path2 such that it is closest to the seam of path1, and so forth.

See image:

If this would also take into account the previous path, it would change also the first path (path0) to be closest to path1 (see poorly drawn image). But then it would not change anything in the further seam changing, right? Still, it could be good to also consider the first path but it does not change so much right? Perhaps we should discuss in person so that it's clearer.

from compas_slicer.

Closed it for now

from compas_slicer.

The way it is made right now, it doesn't fix the start point of the first path. See the example in the image below (coming from example_1).

from compas_slicer.

Yes you are right! I'll fix this asap.

from compas_slicer.