dbekaert / RAiDER / f4ecb7a0-11e6-4c4e-b84a-73358fe56f6c

54%

dev: 53%

Build Type

Pull #810

circleci

Committed by jlmaurer

Commit Message

Route the single-date CDS download through the batch writer

The rebase onto #808 left _get_from_cds still issuing one merged request for
lnsp/z/t/q, which cannot work: CDS returns a mixed-level request as separate
files, and where a single file does come back the surface fields are padded onto
the full 137-level axis so .squeeze() no longer yields the (lat, lon) arrays
calcgeoh and _makeDataCubes expect.

_batch_get_from_cds already splits the request correctly and documents why, so
rather than repeat that logic, treat a single date as a one-element batch. Both
level types now share one download path, which is what stops the two from
drifting apart again -- the pl path was already delegating.

Also restores the latitude fix from #811 in _load_model_level. It was reverted
by resolving the rebase conflict in favour of this branch's copy of ecmwf.py,
which predates that fix; test_ecmwf_levels.py caught it.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

Pull Request Pull Request #810: Multiday download from ECMWF CDS API

Coverage Stats

107 of 136 new or added lines in 12 files covered. (78.68%)

21 existing lines in 4 files now uncovered.

3575 of 6597 relevant lines covered (54.19%)

0.54 hits per line