Skip to content

[codex] Add bounds selection support#3

Merged
TheGreatAlgo merged 3 commits into
mainfrom
codex/add-bounds-selection
Jun 10, 2026
Merged

[codex] Add bounds selection support#3
TheGreatAlgo merged 3 commits into
mainfrom
codex/add-bounds-selection

Conversation

@0xSwego

@0xSwego 0xSwego commented Jun 9, 2026

Copy link
Copy Markdown
Collaborator

Summary

Adds first-class rectangular lon/lat bounds support to GeoSelectionOptions so callers can use client.selectDataset({ selection: { bounds: [west, south, east, north] } }) instead of manually loading and calling GeoTemporalDataset.rectangle().

Changes

  • Adds tuple and object forms for bounds plus optional coordinate key overrides.
  • Applies bounds inside GeoTemporalDataset.select() through the existing rectangle selection implementation.
  • Rejects ambiguous selections that specify both point and bounds.
  • Updates the fake dataset helper to support array coordinate selections used by rectangle tests.
  • Documents the selectDataset bounds usage.

Validation

  • npx vitest run tests/selection-options.test.ts
  • npm run build

@0xSwego 0xSwego marked this pull request as ready for review June 9, 2026 15:34
@TheGreatAlgo TheGreatAlgo merged commit 437fe25 into main Jun 10, 2026
3 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