Skip to content

Bbox support when converting #270

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

Open
wants to merge 11 commits into
base: main
Choose a base branch
from

Conversation

felix-schott
Copy link
Contributor

@felix-schott felix-schott commented Mar 27, 2025

This PR adds support for bbox data when converting between formats:

geojson -> geoparquet

  • When running gpq convert input.geojson output.parquet, bbox information present in the geojson data (top level "bbox" field as per the geojson spec) gets respected and put in a geoparquet bbox column (with the correct covering metadata written to the parquet header)
  • Additionally, this PR introduces a parameter --add-bbox to gpq convert which results in a bbox column being added to the resulting parquet file even if not already present in the input data

geoparquet -> geojson

  • When converting from geoparquet to geojson, the result geojson features will have a correctly populated bbox field (if present in the input parquet file)

Note: this PR depends on #264 which should be merged first (really this should probably be a PR to merge into 100_extract rather than main - the comparison between main and this branch will include all changes of #264 until #264 has been merged). As said before - I'm very happy to try and split this all up into smaller PRs if that makes things easier. These PRs are just the outcome of me tackling some issues without putting too much thought into splitting the work into digestable PRs.

Cheers
Felix

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.

1 participant