Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GeoInteface.getgeom(polygon, i) is very slow. We need a cache for calculating holes and rings only once. #108

Open
rafaqz opened this issue Apr 1, 2024 · 0 comments

Comments

@rafaqz
Copy link
Member

rafaqz commented Apr 1, 2024

Using getgeom(multipolygon) rather than getgeom(polygon, i) is recommended... but in practice the later is used a lot and is currently insanely slow if you have e.g. 100 holes in a polygon.

We could maybe add a cache of bit indices for each polygon with the exterior then all the holes. Then we can just index with it to get each polygon after the first time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant