Skip to content

feat(vortex-spatial): preserve GeoArrow metadata - #9363

Open
HarukiMoriarty wants to merge 1 commit into
developfrom
nemo/geo-metadata
Open

feat(vortex-spatial): preserve GeoArrow metadata#9363
HarukiMoriarty wants to merge 1 commit into
developfrom
nemo/geo-metadata

Conversation

@HarukiMoriarty

@HarukiMoriarty HarukiMoriarty commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

Rationale for this change

Vortex preserved GeoArrow crs metadata but dropped crs_type and edges, making Arrow round-trips lossy.

What changes are included in this PR?

  • Add crs_type and edges to SpatialMetadata with backward-compatible protobuf encoding.
  • Preserve all three fields through spatial Arrow import/export and WKB conversion.
  • Move shared metadata and geometry helpers out of extension/mod.rs.
  • Add metadata serialization and GeoArrow round-trip tests.

What APIs are changed? Are there any user-facing changes?

Exports CrsType and Edges. Spatial computation is unchanged; geo kernels remain planar.

Signed-off-by: Nemo Yu <zyu379@wisc.edu>
@HarukiMoriarty HarukiMoriarty added the changelog/feature A new feature label Aug 11, 2026
@HarukiMoriarty
HarukiMoriarty requested a review from gatesn August 11, 2026 17:30
@codspeed-hq

codspeed-hq Bot commented Aug 11, 2026

Copy link
Copy Markdown

Merging this PR will not alter performance

✅ 1962 untouched benchmarks
⏩ 89 skipped benchmarks1


Comparing nemo/geo-metadata (1ec22c0) with develop (ca7f626)

Open in CodSpeed

Footnotes

  1. 89 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

@HarukiMoriarty HarukiMoriarty changed the title feat(vortex-spatial): preserve GeoArrow metadata feat: preserve GeoArrow metadata and add dense union encoding Aug 11, 2026
@HarukiMoriarty HarukiMoriarty changed the title feat: preserve GeoArrow metadata and add dense union encoding feat(vortex-spatial): preserve GeoArrow metadata Aug 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog/feature A new feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant