dbekaert / RAiDER
51%
dev: 55%

LAST BUILD BRANCH: pr/multiday_download
DEFAULT BRANCH: dev
Repo Added
Files 53
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

LAST BUILD ON BRANCH pr/xarray_memory
branch: pr/xarray_memory
CHANGE BRANCH
x

coverage: 51.261% (+0.1%) from 51.138%
5ecf2d10-e77a-4f5a-b2f4-449f70db4102

Pull #808

circleci

jlmaurer 
Revert the merged ECMWF CDS request and pin the download layout with tests

The single combined request for lnsp/z/t/q could not work. ERA-5 archives lnsp
and z at model level 1 only, so a mixed-level request is not something CDS will
return as one netCDF -- the multi-date path added in #810 splits the request for
exactly this reason. Where a single file is returned at all, the surface fields
come back padded onto the full 137-level axis, so .squeeze() no longer collapses
the level dimension and calcgeoh raises:

  ValueError: could not broadcast input array from shape (137,11,11) into (11,11)

reproduced against test/weather_files/ERA-5_2020_01_30_T13_52_45.nc, which was
downloaded by the pre-split single-request path and still carries that layout.

Restore the two requests and keep the rest of the cleanup: the full-cube
shutil.copy is dead (to_netcdf overwrites out_path) and is dropped.

Nothing in CI performs a CDS retrieval, which is why _get_from_cds had no cover
at all. test_ecmwf_fetch.py stubs cdsapi.Client and drives the real
post-processing into the real reader, asserting the layout contract the two
share: t/q/z as (time, level, lat, lon) cubes and lnsp recovered as lnsp[0, 0].
The stub refuses a mixed-level request, so re-merging the two retrievals fails
the suite instead of failing at download time. Its synthetic responses use the
valid_time and model_level coordinate names the current CDS returns.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Pull Request #808: Update xarray usage in RAiDER to use lazy loading

33 of 58 new or added lines in 5 files covered. (56.9%)

3313 of 6463 relevant lines covered (51.26%)

0.51 hits per line

Relevant lines Covered
6463 RELEVANT LINES 3313 COVERED LINES
0.51 HITS PER LINE
Source Files on pr/xarray_memory

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
5ecf2d10... pr/xarray_memory Revert the merged ECMWF CDS request and pin the download layout with tests The single combined request for lnsp/z/t/q could not work. ERA-5 archives lnsp and z at model level 1 only, so a mixed-level request is not something CDS will return as on... Pull #808 jlmaurer circleci
51.26
ac77e91c... pr/xarray_memory Revert the merged ECMWF CDS request and pin the download layout with tests The single combined request for lnsp/z/t/q could not work. ERA-5 archives lnsp and z at model level 1 only, so a mixed-level request is not something CDS will return as on... Pull #808 jlmaurer circleci
51.26
18f513ef... pr/xarray_memory Revert the merged ECMWF CDS request and pin the download layout with tests The single combined request for lnsp/z/t/q could not work. ERA-5 archives lnsp and z at model level 1 only, so a mixed-level request is not something CDS will return as on... Pull #808 jlmaurer circleci
51.26
18512a66... pr/xarray_memory Bug-fix of dataset variable assignment Pull #808 sssangha circleci
50.1
061ef4af... pr/xarray_memory Potential Resource Leak if open_dataset Fails Mid-Loop Pull #808 sssangha circleci
50.1
c43c7b46... pr/xarray_memory update CHANGELOG Pull #808 jlmaurer circleci
50.09
See All Builds (1247)