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

Update Obs4MIPs datasets names (ESGF) in ECS recipes #3327

Merged
merged 4 commits into from Nov 9, 2023

Conversation

rbeucher
Copy link
Contributor

Description

Following discussion here #2974

CERES-EBAF Ed2-7 is not available on ESGF anymore.
CERES-EBAF Ed2-8 is available for variable rlut, rlutcs, rsdt, rsut, rsutcs.
This PR removes the version facet for CERES-EBAF which fix automatic download from ESGF.

recipe_ecs_constraints.yml
AIRS RetStd-v5 ['hus'] >> AIRS-2-1
AIRS-2-0 v2 ['hur'] >> AIRS-2-0
CERES-EBAF Ed2-7 ['rsdt', 'rsut', 'rsutcs'] >> CERES-EBAF
GPCP-SG v2.2 ['pr'] >> GPCP-V2.2

recipe_ecs_scatter.yml
AIRS RetStd-v5 ['hus', 'husStderr'] >> AIRS-2-1
CERES-EBAF Ed2-7 ['rsdt', 'rsut', 'rsutcs'] >> CERES-EBAF
TRMM-L3 7A ['pr', 'prStderr'] >> TRMM


Before you get started

Checklist

It is the responsibility of the author to make sure the pull request is ready to review. The icons indicate whether the item will be subject to the 🛠 Technical or 🧪 Scientific review.

New or updated recipe/diagnostic


To help with the number of pull requests:

Copy link
Member

@bouweandela bouweandela left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In recipe_ecs_scatter.yml, the reference_datasets still need to be updated to use TRMM.

@bouweandela
Copy link
Member

Thanks for the updates! Could you please merge the main branch from this GitHub git repository into your branch? That should solve the failing tests. For example, if you have added this repository under the name upstream, you can do this by running the command git pull upstream main.

@rbeucher
Copy link
Contributor Author

rbeucher commented Nov 8, 2023

Sorry, I forgot to update our main branch

@zklaus zklaus dismissed bouweandela’s stale review November 9, 2023 16:17

Comments have been addressed.

@zklaus zklaus merged commit d14c309 into ESMValGroup:main Nov 9, 2023
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants