Moved dkrz-era5 data source in DKRZ default config to separate file#2996
Moved dkrz-era5 data source in DKRZ default config to separate file#2996bouweandela merged 4 commits intomainfrom
dkrz-era5 data source in DKRZ default config to separate file#2996Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2996 +/- ##
==========================================
- Coverage 95.69% 95.68% -0.01%
==========================================
Files 267 267
Lines 15726 15726
==========================================
- Hits 15049 15048 -1
- Misses 677 678 +1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
How about moving this configuration to a separate file, e.g. One of the issues we had with the old configuration, which contained lots of commented out settings, was that it was unclear to new users which bits they needed to uncomment, resulting in syntax errors. |
|
Very good point, completely missed that we can do that now! 😅 Changed in 0b987dc. |
dkrz-era5 data source in DKRZ default configdkrz-era5 data source in DKRZ default config to separate file
|
Thanks! The docs will also need to be updated: ESMValCore/doc/quickstart/find_data.rst Lines 140 to 166 in 974ff8c |
|
Docs updated in 279bc5e |
Description
To avoid errors like
Dataset key 'grib_id' must be specifiedwhen using the default DKRZ configuration with published recipes.See #2995.
Docs: https://esmvaltool--2996.org.readthedocs.build/projects/ESMValCore/en/2996/quickstart/find_data.html#era5-in-grib-format-available-on-dkrz-s-levante-or-downloaded-from-the-cds
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.
To help with the number pull requests: