meerk40t / svgelements

SVG Parsing for Elements, Paths, and other SVG Objects.
MIT License
133 stars 29 forks source link

Fixed implementation of CubicBeizer._length_scipy #62

Closed abey79 closed 3 years ago

abey79 commented 3 years ago

Implementation of #58 was naive. The callback provided to scipy's quad was way to slow due to the getitem, etc. I rewrote the callback inline in a way that makes it much more efficient.

tatarize commented 3 years ago

Also, much tighter looking code and generally prettier. Way easier to understand what's going on. This from code that started getting so shifty and spaghetti-like that I gave up.