read should (optionally?) crop geometries to the SVG bounds
#94
Comments
This was referenced Nov 24, 2020
Merged
abey79
added a commit
that referenced
this issue
Nov 25, 2020
SVG read operation have been massively improved by replacing svgpathtools with svgelements and rewriting the parsing operations. Thanks a lot to @tatarize for his support in the process. `read` command: * Execution time drastically reduced in most cases. * Improved robustness to various aspects of the SVG spec. * Now geometries are cropped to the SVG dimension (can be disabled with `--no-crop`). * Line simplification is no longer implicit and must be enabled with `--simplify` * Line simplification is now applied only on curved elements (circle, arc, bezier, etc.) and element-by-element. Polygon/polylines are not affect by the simplification (use the `linesimplify` command if this is needed) * Multiprocessed SVG parsing is now optionally available (`--parallel`). It should be only enabled for SVG with many curvy elements and when using `--simplify`. Other situation show no improvements. * Proper support for SVG with missing or percentage width/height. Use the `--display-size` and `--display-landscape` options to specify the default size (corresponding to 100% width/height). * Fixed error on with 0-radius arc elements `write` command: * Removed the rarely used `--single-path` option. * Now uses <line>, <polyline> and <polygon> instead of <path>, which improves read time a lot if these SVG are reused in vpype. Minor fixes: * mypy added to pyproject.toml * Added test for non-duplicate consecutive points * Fixed test_file_* which wouldn't actually fail on error * added many test SVG files * test SVG files prefixed with _ are now ignored Fixes #21 #31 #53 #71 #77 #87 #94
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
No description provided.
The text was updated successfully, but these errors were encountered: