Skip to content

switch to polygon from bbox - #10

Merged
annefou merged 3 commits into
FAIR2Adapt:mainfrom
tinaok:main
May 29, 2026
Merged

switch to polygon from bbox#10
annefou merged 3 commits into
FAIR2Adapt:mainfrom
tinaok:main

Conversation

@tinaok

@tinaok tinaok commented May 29, 2026

Copy link
Copy Markdown
Collaborator

This pull request replaces the bbox-based HEALPix coverage with a polygon-based coverage using user-defined vertices.

Because HEALPix uses hierarchical spherical cells, the selected storage area does not need to be constrained to a rectangular bounding box. Using polygon vertices allows us to describe the area of interest more precisely, especially for coastal regions with irregular shapes.

As a result, fewer irrelevant cells are selected outside the target region, which helps reduce disk storage usage while preserving the required spatial coverage.

@tinaok
tinaok requested a review from annefou May 29, 2026 04:54
annefou and others added 2 commits May 29, 2026 12:01
Polygon, box and gpd are already imported in the earlier cell, and
the remaining cartopy/geopandas/matplotlib imports are now sorted.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Whitespace-only change to satisfy `ruff format --check`.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@annefou
annefou merged commit 4d2409d into FAIR2Adapt:main May 29, 2026
5 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