Skip to content

Fix experiment selection for computing ZEC#589

Open
bouweandela wants to merge 1 commit intoClimate-REF:mainfrom
bouweandela:fix-zec-dataset-experiment
Open

Fix experiment selection for computing ZEC#589
bouweandela wants to merge 1 commit intoClimate-REF:mainfrom
bouweandela:fix-zec-dataset-experiment

Conversation

@bouweandela
Copy link
Contributor

@bouweandela bouweandela commented Mar 9, 2026

Description

Select the correct experiment for computing ZEC.

Documentation at https://docs.esmvaltool.org/en/latest/recipes/recipe_zec.html

Follow up to #585, closes #586.

Checklist

Please confirm that this pull request has done the following:

  • Tests added
  • Documentation added (where applicable)
  • Changelog item added to changelog/

@bouweandela bouweandela force-pushed the fix-zec-dataset-experiment branch from 1f14ddb to ba9aa9c Compare March 9, 2026 09:13
@codecov
Copy link

codecov bot commented Mar 9, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

Flag Coverage Δ
core 93.18% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@bouweandela bouweandela force-pushed the fix-zec-dataset-experiment branch from ba9aa9c to eec3610 Compare March 9, 2026 10:13
@bouweandela bouweandela marked this pull request as ready for review March 9, 2026 10:47
@bouweandela
Copy link
Contributor Author

Opened #590 for the unrelated failing test.

facets={
"branded_variable": "tas_tavg-h2m-hxy-u",
"experiment_id": experiments,
"experiment_id": "esm-flat10-zec",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is the experiment for CMIP7, but the current scheme we have for mocking up CMIP7 data does not convert experiments, so this leads to an empty solve.

@bouweandela bouweandela requested a review from lewisjared March 9, 2026 10:59
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 this pull request may close these issues.

ZEC diagnostic: consider supporting esm-1pctCO2 as a reference experiment

1 participant