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

h3_cells_to_multi_polygon_wkt error #137

Open
samlalwani opened this issue Dec 2, 2024 · 2 comments
Open

h3_cells_to_multi_polygon_wkt error #137

samlalwani opened this issue Dec 2, 2024 · 2 comments

Comments

@samlalwani
Copy link

On calling ST_GeomFromText over the column generated by h3_cells_to_multi_polygon_wkt ,
I see the following error : shapely.errors.GEOSException: IllegalArgumentException: Points of LinearRing do not form a closed linestring

@isaacbrodsky isaacbrodsky mentioned this issue Dec 2, 2024
@isaacbrodsky
Copy link
Owner

Thanks for reporting this. I am not able to immediately reproduce because using ST_GeomFromText within DuckDB is fine with the multipolygons not being closed. That being said, it looks like this function does indeed need to duplicate the first vertex to be compliant with how OGC wants the WKT. Please see #138

@samlalwani
Copy link
Author

samlalwani commented Dec 2, 2024

I was calling ST_GeomFromText within DuckDB , its strange you did not observe this issue. Will try to create a test dataset.

I moved this operation to geopandas using h3.cells_to_h3shape which works perfectly though I loose the processing/scalabale advantage of dudkdb.

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

2 participants