Opened 7 years ago
Last modified 7 years ago
#28020 closed Cleanup/optimization
make GEOSGeometry.json use OGRGeometry.json again (for performance reasons) — at Initial Version
| Reported by: | Sergey Fedoseev | Owned by: | nobody |
|---|---|---|---|
| Component: | GIS | Version: | dev |
| Severity: | Normal | Keywords: | |
| Cc: | Triage Stage: | Ready for checkin | |
| Has patch: | yes | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description
GEOSGeometry.json was using OGRGeometry.json prior to 1.9. That was changed in #25141. Now GDAL is required (#26753) and I believe we should restore previous GEOSGeometry.json implementation for performance reasons:
In [67]: ls = LineString([(x, x) for x in range(1000)]) In [68]: %timeit a = ls.json 10 loops, best of 3: 42.6 ms per loop In [69]: %timeit a = ls.ogr.json 100 loops, best of 3: 3.75 ms per loop
Note:
See TracTickets
for help on using tickets.