Skip to content

feat(vortex-geo): add Hilbert scalar function - #9239

Open
HarukiMoriarty wants to merge 2 commits into
nemo/geo-intersection-nativefrom
nemo/geo-hilbert-bounds
Open

feat(vortex-geo): add Hilbert scalar function#9239
HarukiMoriarty wants to merge 2 commits into
nemo/geo-intersection-nativefrom
nemo/geo-hilbert-bounds

Conversation

@HarukiMoriarty

@HarukiMoriarty HarukiMoriarty commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add vortex.geo.hilbert for native geometry columns with constant Rect bounds
  • reuse the existing GeoEnvelope kernel and unary constant/column dispatch
  • return nullable u32 Hilbert keys for spatial clustering
  • add point and multipolygon Divan benchmarks

@HarukiMoriarty HarukiMoriarty added the changelog/feature A new feature label Aug 6, 2026
@HarukiMoriarty
HarukiMoriarty force-pushed the nemo/geo-hilbert-bounds branch from 00fa457 to c0d4338 Compare August 6, 2026 14:41
@codspeed-hq

codspeed-hq Bot commented Aug 6, 2026

Copy link
Copy Markdown

Merging this PR will not alter performance

✅ 1921 untouched benchmarks
🆕 2 new benchmarks
⏩ 43 skipped benchmarks1

Performance Changes

Mode Benchmark BASE HEAD Efficiency
🆕 Simulation multipolygons N/A 1.4 ms N/A
🆕 Simulation points N/A 201.9 µs N/A

Comparing nemo/geo-hilbert-bounds (c0d4338) with nemo/geo-intersection-native (f2b5e90)

Open in CodSpeed

Footnotes

  1. 43 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 force-pushed the nemo/geo-hilbert-bounds branch from c0d4338 to 2bf6598 Compare August 6, 2026 17:58
@HarukiMoriarty
HarukiMoriarty force-pushed the nemo/geo-hilbert-bounds branch from 2bf6598 to 1918865 Compare August 6, 2026 20:13
Signed-off-by: Nemo Yu <zyu379@wisc.edu>
Signed-off-by: Nemo Yu <zyu379@wisc.edu>
@HarukiMoriarty
HarukiMoriarty force-pushed the nemo/geo-hilbert-bounds branch from 1918865 to 1d52139 Compare August 6, 2026 20:16
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