'Brim Replace Support' did not replace support-roof.
Models that only had a little gap between the BP and the model had support roof instead of support infill where the brim should go with the 'Brim Replaces Support' option on. So now we take the support roof (interface) into account as well.
This was a 5 micron lenience in the simplification function that allowed the simplification to recognise colinear segments and remove the intermediate vertex if they are colinear. This had a bit of lenience because if the line segment is diagonal there is pretty much never an exactly straight line due to vertices being rounded to 1 micron coordinates. They will shift by up to a micrometre. So the 5 micron lenience was a bit too much and would cause the polygons to be simplified a bit too much if the user requests a very high resolution. Instead of 5um we now just take 1um.
In commit 6ed16d57c161ed8cf4ef80fb4fba7f515ae3c52e the Spaghetti Infill routine was removed here, for which the indentation was changed. There was a mistake there with how the brackets were read. This line should also have been unindented. It belongs to the normal infill generation routine, not to the Sierpinski fractal pre-computation.
Fixes CURA-7613.
f5c81e2...
by
Jelle Spijker <email address hidden>
Merge pull request #1298 from Ultimaker/CURA-6684_Fix_missing_skin_infill_in_roofing_layers
CURA-7614: Fix missing skin infill in roofing layers
