Skip to content

chore: Bump geodatasets to 2026.5.1 to fix nybb download in CI#1207

Merged
kylebarron merged 1 commit into
mainfrom
bump-geodatasets
Jun 12, 2026
Merged

chore: Bump geodatasets to 2026.5.1 to fix nybb download in CI#1207
kylebarron merged 1 commit into
mainfrom
bump-geodatasets

Conversation

@kylebarron

@kylebarron kylebarron commented Jun 12, 2026

Copy link
Copy Markdown
Member

this should unblock the CI failure we currently have in #1206 , #1205 , #1204

Written by claude:


Problem

CI fails on 13 tests with requests.exceptions.HTTPError: 403 Client Error: Forbidden for https://www.nyc.gov/assets/planning/download/zip/data-maps/open-data/nybb_16a.zip — nyc.gov no longer serves the nybb test dataset. Local runs only pass when a warm geodatasets cache exists.

Fix

Bump geodatasets 2026.1.0 → 2026.5.1 in uv.lock (single-package upgrade, nothing else changes). The new release relocates the nybb download to https://github.com/geopandas/geodatasets/raw/refs/heads/main/data_backup/nybb_16a.zip.

Verification

With the geodatasets cache removed (simulating CI), tests/test_map.py::test_default_view_state_inferred and tests/test_viz.py::test_viz_reproject — both failing in CI — pass and download from the new URL: 2 passed.

This unblocks CI for #1205 and #1206, which should be rebased/rerun once this merges.

🤖 Generated with Claude Code

nyc.gov now returns 403 Forbidden for the nybb dataset, failing 13
tests in CI (any environment without a warm geodatasets cache).
geodatasets 2026.5.1 moved the nybb download to the geodatasets
repository's data backup on GitHub.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@ds-release-bot ds-release-bot Bot added the chore label Jun 12, 2026
@kylebarron
kylebarron enabled auto-merge (squash) June 12, 2026 16:37
@kylebarron
kylebarron merged commit edf308e into main Jun 12, 2026
7 checks passed
@kylebarron
kylebarron deleted the bump-geodatasets branch June 12, 2026 16:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant