Skip to content

Update example file for GeoParquet 2.0 - #280

Merged
cholmes merged 19 commits into
opengeospatial:mainfrom
paleolimbot:parquet-2-0-example-files
Jun 1, 2026
Merged

Update example file for GeoParquet 2.0#280
cholmes merged 19 commits into
opengeospatial:mainfrom
paleolimbot:parquet-2-0-example-files

Conversation

@paleolimbot

Copy link
Copy Markdown
Collaborator

Just a draft until #278 merges! This PR updates the example file to (1) write Parquet GEOMETRY as the storage type and (2) a few nits I noticed in the file.

@paleolimbot
paleolimbot force-pushed the parquet-2-0-example-files branch from cc347ec to fb58beb Compare February 11, 2026 22:31
@cholmes

cholmes commented Feb 12, 2026

Copy link
Copy Markdown
Member

Is this ready for merge? Looks like you fixed it up, but it's still marked 'draft'.

@cholmes

cholmes commented Feb 12, 2026

Copy link
Copy Markdown
Member

Just tried it out - thanks for doing this. It looks to be working, though geoparquet-io didn't pick up a geometry type from the Parquet metadata. If you set it let me know and I can fix my code, if you didn't set that it would be good to.

I'd say feel free to merge. I do think we should have a wider set of example data, showing projections and geography at the very least. And our test_data directory still includes arrow types, so we should get rid of those / update that script to use the stuff here. If I find time I can likely do that, but I'm slammed for the next bit.

@paleolimbot
paleolimbot force-pushed the parquet-2-0-example-files branch from e19166a to 0245bfa Compare April 6, 2026 17:59
@paleolimbot
paleolimbot marked this pull request as ready for review April 6, 2026 21:29
@paleolimbot

Copy link
Copy Markdown
Collaborator Author

@cholmes Apologies for taking an inappropriately long time to circle back here, but I think this implements your suggestions (minus the extra test data).

though geoparquet-io didn't pick up a geometry type from the Parquet metadata

I added a test here to make sure it's where I think it is in the Parquet statistics...perhaps this has been cleared up in geoparquet-io by now (or the test can help fix it).

I do think we should have a wider set of example data, showing projections and geography at the very least

I'll do a release of geoarrow-data when we get around to a release of the spec...the tooling is much better now than the last time I did a release and it should be easy to get good examples of various GeoParquet versions for all the different artifacts.

@cholmes

cholmes commented Jun 1, 2026

Copy link
Copy Markdown
Member

@copilot resolve the merge conflicts in this pull request

@cholmes cholmes left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great, going to merge. I may take a crack at more extensive test date, that exercises more of the fields, but will do that in a separate PR.

@cholmes
cholmes merged commit bef58ca into opengeospatial:main Jun 1, 2026
2 checks passed
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