Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Soil moisture requests failing due to CDS API changes #68

Closed
pont-us opened this issue May 3, 2023 · 1 comment · Fixed by #69
Closed

Soil moisture requests failing due to CDS API changes #68

pont-us opened this issue May 3, 2023 · 1 comment · Fixed by #69

Comments

@pont-us
Copy link
Member

pont-us commented May 3, 2023

The CDS server has recently changed the API for the soil moisture dataset: the valid version specifiers are no longer

'v201706.0.0', 'v201812.0.0', 'v201812.0.1', 'v201912.0.0'

but

'v201706', 'v201812', 'v201912', 'v202012',

This results in an error when an old-style version specifier is supplied (or when none is supplied and the default is used), and new-style version specifiers can't be used since they are checked against the old JSON Schema.

Steps to reproduce:

from xcube.core.store import new_data_store
cds = new_data_store('cds')
ds = cds.open_data(
    'satellite-soil-moisture:volumetric:monthly',
    time_range=['2015-01-01', '2015-12-31']
)

Result:

Exception: the request you have submitted is not valid. Value 'v201912.0.0' not valid for parameter 'version', valid values are: v201706, v201812, v201912, v202012.

The variable names have changed too.

@pont-us
Copy link
Member Author

pont-us commented May 5, 2023

On further inspection, the CDS backend API parameter schemata have also changed for the ERA5 and satellite sea ice thickness datasets. Further updates to xcube-cds will be needed.

These changes weren't picked up by the unit tests because the tests use mock cdsapi clients with pre-populated data rather than the live API.

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 a pull request may close this issue.

1 participant