Skip to content

fix(mcdwd): offer only the days asked for, not a composite starting the day before - #26

Merged
turban merged 1 commit into
mainfrom
fix/mcdwd-periods-in-range
Oct 20, 2026
Merged

turban merged 1 commit into
mainfrom
fix/mcdwd-periods-in-range

Conversation

@turban

@turban turban commented Oct 20, 2026

Copy link
Copy Markdown
Collaborator

Ingesting modis_flood_daily failed for any day, with OCS answering Source cannot materialize the requested temporal scope from 2026-10-15; first available is 2026-10-14.

The product is a 2-day composite, so the granule starting on the 14th overlaps the 15th, and the CMR search for the 15th returns it. periods() listed every granule's start day, so it offered the 14th for a request for the 15th, and OCS refuses a source that starts before the requested day. periods() now keeps only the days within the requested range; fetch_period already filtered by day.

Found ingesting a sample of every data source on the Nepal test instance. A new test returns a composite starting the day before and checks only the requested day is offered; it fails without the fix.

@turban
turban merged commit 6ec5811 into main Oct 20, 2026
1 check 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.

1 participant