Skip to content

Anchor time filters to shared dataset coverage - #44

Open
raederhans wants to merge 1 commit into
mainfrom
codex/dataset-anchored-time-window
Open

Anchor time filters to shared dataset coverage#44
raederhans wants to merge 1 commit into
mainfrom
codex/dataset-anchored-time-window

Conversation

@raederhans

Copy link
Copy Markdown
Owner

Summary

  • anchor default and recent Crime windows to the latest month shared by live CARTO data and the validated tract snapshot
  • preserve the true live coverage maximum for provenance and valid shared URLs
  • disable recent presets until coverage is ready, with no wall-clock fallback

Verification

  • VITE_FEATURE_DIARY=1 VITE_TRACT_CRIME_SNAPSHOT=1 npm run validate
  • VITE_FEATURE_DIARY=1 VITE_TRACT_CRIME_SNAPSHOT=1 npm run test:browser-smoke
  • npm audit --audit-level=high

Evidence

Live coverage was 2006-01-01 through 2026-08-02 while the shipped tract snapshot covers 2025-08-01 through 2026-08-01. The browser regression now proves the default remains 2025-08 instead of drifting to 2025-09 when live coverage crosses the month boundary.

Live CARTO coverage can advance into a new month before the validated tract snapshot. Preserve the live maximum for provenance, but anchor defaults and recent presets to the latest date shared with the snapshot; never substitute the wall clock while coverage is unavailable.

Constraint: Existing valid shared URLs remain usable within live coverage.

Rejected: Replace coverageMax with the snapshot maximum | this would understate live API coverage and reject valid live-only windows.

Confidence: high

Scope-risk: narrow

Reversibility: clean

Directive: Keep windowAnchorMax separate from coverageMax when adding data sources.

Tested: VITE_FEATURE_DIARY=1 VITE_TRACT_CRIME_SNAPSHOT=1 npm run validate; npm run test:browser-smoke; npm audit --audit-level=high
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