Skip to content

Conversation

@kylebarron
Copy link
Member

@kylebarron kylebarron commented Apr 15, 2025

Change list

cc @paleolimbot

@kylebarron kylebarron merged commit 7109bcd into main Apr 15, 2025
21 checks passed
@kylebarron kylebarron deleted the kyle/geoarrow-test-crate branch April 15, 2025 03:56
@kylebarron
Copy link
Member Author

Well, this doesn't actually work out as is because geoarrow-test depends on geoarrow-array, but we want to use this data from within the geoarrow-array tests, which would cause a circular dependency. So we'll have to move this test data into geoarrow-array after all

@kylebarron
Copy link
Member Author

And we can't convert to geo_types anyways, because geo_types only supports 2D data and doesn't support empty points. So we need to update the included wkt strings to be f64 type, and return Wkt objects.

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

Successfully merging this pull request may close these issues.

2 participants