Skip to content

[FEATURE] Download shape [#310]#323

Merged
Brandon Liu (bdon) merged 1 commit into
mainfrom
download-shape
Jul 10, 2026
Merged

[FEATURE] Download shape [#310]#323
Brandon Liu (bdon) merged 1 commit into
mainfrom
download-shape

Conversation

@bdon

@bdon Brandon Liu (bdon) commented Jul 7, 2026

Copy link
Copy Markdown
Collaborator

[FEAT] modify downloaded GeoJSON to match shape of overturemaps-py CLI

  • lift id field to top-level of GeoJSON feature
  • remove bbox property
  • emit Polygon/LineString instead of Multi- geometries

Fixes #310

@bdon Brandon Liu (bdon) changed the title [FEAT] Download shape [#310] [FEATURE] Download shape [#310] Jul 7, 2026
… CLI

* lift id field to top-level of GeoJSON feature
* remove bbox property
* emit Polygon/LineString instead of Multi- geometries

Fixes #310

Signed-off-by: Brandon Liu <bdon@bdon.org>
@github-actions

github-actions Bot commented Jul 7, 2026

Copy link
Copy Markdown

🗺️ Overture Explorer preview is live!

🌍 Preview https://staging.overturemaps.org/explore-site/pr/323/index.html
🕐 Updated 2026-07-07T19:21:58Z
📝 Commit ab1839f

Note

♻️ This preview updates automatically with each push to this PR.

@bdon Brandon Liu (bdon) marked this pull request as ready for review July 9, 2026 18:57
@bdon Brandon Liu (bdon) requested a review from a team as a code owner July 9, 2026 18:57
@bdon

Copy link
Copy Markdown
Collaborator Author

John McCall (@lowlydba) take a look when you can!

@lowlydba John McCall (lowlydba) left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Nice, thank you! I sort of arbitrarily defined the download structure, so I feel better knowing it has partity with the CLI now 😁

@bdon Brandon Liu (bdon) merged commit 5dbd70e into main Jul 10, 2026
13 of 14 checks passed
@bdon Brandon Liu (bdon) deleted the download-shape branch July 10, 2026 17:27
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.

Download button: make sure data shape is consistent with DuckDB and Python downloads

3 participants