Skip to content

Added a CMIP7 version for recipe_ref_trend_regions.yml, recipe_ref_tannual_cycle_region.yml and recipe_ref_timeseries_region.yml#4460

Merged
bouweandela merged 12 commits into
mainfrom
update_recipe_ref_trend_regions_for_CMIP7
May 28, 2026
Merged

Added a CMIP7 version for recipe_ref_trend_regions.yml, recipe_ref_tannual_cycle_region.yml and recipe_ref_timeseries_region.yml#4460
bouweandela merged 12 commits into
mainfrom
update_recipe_ref_trend_regions_for_CMIP7

Conversation

@katjaweigel
Copy link
Copy Markdown
Contributor

@katjaweigel katjaweigel commented May 15, 2026

Add recipe_ref_trend_regions_cmip7.yml

Description

Added a CMIP7 version for recipe_ref_trend_regions.yml
CMIP7 test data can be found under:
/work/bd0854/DATA/ESMValTool2/CMIP7_TEST/MIP-DRS7/CMIP7/CMIP/MOHC/DUMMY-MODEL/historical
(historical extended with ssp126 to reach 2024)

Updates OBS data: pr to GPCP V3.3 and hus to obs4MIPs version of ERA-5


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:

Add recipe_ref_trend_regions_cmip7.yml
@katjaweigel katjaweigel changed the title Add files via upload Added a CMIP7 version for recipe_ref_trend_regions.yml May 15, 2026
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented May 15, 2026

CLA assistant check
All committers have signed the CLA.

@katjaweigel
Copy link
Copy Markdown
Contributor Author

CLA assistant check Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

There is some technical issue, I can get on the page but can't click ok to sign? (with both firefox and edge)

@katjaweigel
Copy link
Copy Markdown
Contributor Author

CLA assistant check Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

There is some technical issue, I can get on the page but can't click ok to sign? (with both firefox and edge)

Now it worked, but the page loads really slow.

@katjaweigel katjaweigel self-assigned this May 15, 2026
@katjaweigel
Copy link
Copy Markdown
Contributor Author

Precipitation (pr) is now the only variable, where ERA-5 data are not available through obs4MIPs (but they are available as "native" netCDF data on Levante until 2021: /work/bd0854/DATA/ESMValTool2/RAWOBS/Tier3/ERA5/v1/mon/pr/); HadCRUT5 (tas) is also Tier2 and available until 2021 on Levante (/work/bd0854/DATA/ESMValTool2/OBS/Tier2/HadCRUT5/)

@katjaweigel katjaweigel changed the title Added a CMIP7 version for recipe_ref_trend_regions.yml Added a CMIP7 version for recipe_ref_trend_regions.yml, recipe_ref_tannual_cycle_region.yml and recipe_ref_timeseries_region.yml May 16, 2026
Include recipe_ref_timeseries_region_cmip7.yml and recipe_ref_annual_cycle_region_cmip7.yml because they need the same changes  as recipe_ref_trend_regions.yml to work for CMIP7
@katjaweigel
Copy link
Copy Markdown
Contributor Author

Adding recipe_ref_timeseries_region_cmip7.yml and recipe_ref_timeseries_region_cmip7.yml because they need the same changes.

Correct GPCP data version in plot_kwargs
Correct GPCP data version in plot_kwargs
@hb326 hb326 added the REF Important for the CMIP Rapid Evaluation Framework (REF) label May 18, 2026
Comment thread esmvaltool/recipes/ref/recipe_ref_trend_regions_cmip7.yml Outdated
Comment thread esmvaltool/recipes/ref/recipe_ref_trend_regions_cmip7.yml
Remove unnecessary blank lines
@katjaweigel katjaweigel requested a review from hb326 May 20, 2026 11:21
@hb326
Copy link
Copy Markdown
Contributor

hb326 commented May 20, 2026

I ran the recipe "recipe_ref_timeseries_region_cmip7.yml" and it worked for me as well with the fake CMIP7 data. The output looked good. So I think this is good to go.

@hb326
Copy link
Copy Markdown
Contributor

hb326 commented May 20, 2026

I tried to run the trend recipe and I ran twice in the same error:
ValueError: "valid_max" is not of a suitable value (36524220.15025164)

This is again connected to the newer version of the GPCP data that we use in the recipe. I have so far no explanation why it would crash THIS recipe, but does not crash one of the other two...

@katjaweigel
Copy link
Copy Markdown
Contributor Author

I tried to run the trend recipe and I ran twice in the same error: ValueError: "valid_max" is not of a suitable value (36524220.15025164)

This is again connected to the newer version of the GPCP data that we use in the recipe. I have so far no explanation why it would crash THIS recipe, but does not crash one of the other two...

Hi @hb326

this should be solved if you use a very recent developement version of the Core, where ESMValGroup/ESMValCore#3072 is already merged?

@hb326
Copy link
Copy Markdown
Contributor

hb326 commented May 20, 2026

I am using the latest development version of ESMValCore. But for whatever reason I run into this error nonetheless. As I said, the other two recipes run fine with the new version of the GPCP data (and the development version of ESMValCore).
@bouweandela: do you think the use of the "trend" preprocessor creates a problem here?

@katjaweigel
Copy link
Copy Markdown
Contributor Author

katjaweigel commented May 20, 2026

@hb326 your're right, now it does not run anymore, the strange thing is, that it did some days ago, see:
/scratch/b/b380216/recipe_ref_trend_regions_cmip7_20260515_165709/

@katjaweigel
Copy link
Copy Markdown
Contributor Author

@hb326 your're right, now it does not run anymore, the strange thing is, that it did some days ago, see: /scratch/b/b380216/recipe_ref_trend_regions_cmip7_20260515_165709/

Testing again I realized, that I did not have the latest version of ESMValCore (anymore) but overwrote my pyproject.toml with the default version which takes the stable Core release, with the latest Core version (and merging the current ESMValTool main) it works?

@hb326 hb326 added this to the v2.15.0 milestone May 26, 2026
@hb326
Copy link
Copy Markdown
Contributor

hb326 commented May 27, 2026

I finally, FINALLY got the trend recipe to run through as well. I had some fights with Pixi, but they seem to be over now.
This recipe is also good to be merged from my perspective. Thanks, @katjaweigel!

Comment thread esmvaltool/recipes/ref/recipe_ref_trend_regions_cmip7.yml Outdated
Comment thread esmvaltool/recipes/ref/recipe_ref_trend_regions_cmip7.yml Outdated
Comment thread esmvaltool/recipes/ref/recipe_ref_trend_regions_cmip7.yml Outdated
Comment thread esmvaltool/recipes/ref/recipe_ref_trend_regions_cmip7.yml Outdated
katjaweigel and others added 5 commits May 28, 2026 13:58
Co-authored-by: Bouwe Andela <b.andela@esciencecenter.nl>
Co-authored-by: Bouwe Andela <b.andela@esciencecenter.nl>
Co-authored-by: Bouwe Andela <b.andela@esciencecenter.nl>
Co-authored-by: Bouwe Andela <b.andela@esciencecenter.nl>
@katjaweigel katjaweigel requested a review from bouweandela May 28, 2026 13:43
@bouweandela bouweandela merged commit f1cb6b5 into main May 28, 2026
9 checks passed
@bouweandela bouweandela deleted the update_recipe_ref_trend_regions_for_CMIP7 branch May 28, 2026 14:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

REF Important for the CMIP Rapid Evaluation Framework (REF)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Testing recipe_ref_trend_regions with CMIP7.

4 participants