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

Fix recipe_check_obs to be aligned with DKRZ #3673

Merged
merged 1 commit into from
Jun 21, 2024
Merged

Conversation

LisaBock
Copy link
Contributor

@LisaBock LisaBock commented Jun 21, 2024

Description

There are missing observations when testing the recipe for the release on DKRZ.

Changes in the recipe:

  • AGCD: Tier 3 -> Tier 2 in the recipe

  • AERONET: version -> '20240406'

  • ERA5 rlus + rsus, 1hr+mon: commented out in the recipe as they are not available at DKRZ

  • JRA-25: changed end date to 2007 (instead of 2013)

  • Closes Missing data on DKRZ for recipe_check_obs.yml #3660


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

@LisaBock
Copy link
Contributor Author

All data is found with the changes but the recipe is still not running successfully for me. @axel-lauer Could you test it as well?

@axel-lauer
Copy link
Contributor

Thanks @LisaBock ! I just submitted a job to test recipe_check_obs.yml.

Copy link
Contributor

@axel-lauer axel-lauer left a comment

Choose a reason for hiding this comment

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

Looks good! Thank you, @LisaBock ! I ran the recipe and it works fine (got an out of memory error at first but no problems when I split the recipe into two parts).

@axel-lauer axel-lauer merged commit 967589c into main Jun 21, 2024
8 checks passed
@axel-lauer axel-lauer deleted the fix_recipe_check_obs branch June 21, 2024 12:39
@ehogan ehogan added this to the v2.11.0 milestone Jun 25, 2024
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.

Missing data on DKRZ for recipe_check_obs.yml
5 participants