proposed patch for Spiro spline LPE
this is a follow-up to LP Bug 415079. https://bugs.launchpad.net/inkscape/+bug/415079
I'd like to propose a patch to deal with occasional (multiple) circular loops in the Spiro Spline LPE. The patch consists of a re-application of a common clamp which is normally applied to the angles, but which appears to need to be re-applied at the final stage of the calculation. I've tested this offline in a standalone calculation to confirm that it works. I wonder if someone would be willing to test this directly in Inkscape.
http://www.nabble.com/file/p25015469/spiro.diff spiro.diff
just writing to propose a second clamp, this time on the variable ks[0]. I believe this should be the end of the constraint proposals, since now both input and output are constrained. Details are in the bug report LP415079.
tia, Alvin http://www.nabble.com/file/p25041539/spiro_patch2.diff spiro_patch2.diff
participants (1)
-
Alvin Penner