Skip to content

Make TCRE diagnostic compatible with CMIP7 data#702

Draft
schlunma wants to merge 13 commits into
Climate-REF:mainfrom
schlunma:tcre_cmip7
Draft

Make TCRE diagnostic compatible with CMIP7 data#702
schlunma wants to merge 13 commits into
Climate-REF:mainfrom
schlunma:tcre_cmip7

Conversation

@schlunma
Copy link
Copy Markdown
Contributor

Description

Addresses #647.

Checklist

Please confirm that this pull request has done the following:

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

@codecov
Copy link
Copy Markdown

codecov Bot commented Jun 1, 2026

Codecov Report

❌ Patch coverage is 80.00000% with 3 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...ool/src/climate_ref_esmvaltool/diagnostics/tcre.py 78.57% 1 Missing and 2 partials ⚠️
Flag Coverage Δ
core 93.11% <ø> (ø)
providers 91.71% <80.00%> (-0.14%) ⬇️

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

Files with missing lines Coverage Δ
...ef-esmvaltool/src/climate_ref_esmvaltool/recipe.py 86.61% <100.00%> (ø)
...ool/src/climate_ref_esmvaltool/diagnostics/tcre.py 93.75% <78.57%> (-6.25%) ⬇️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@schlunma
Copy link
Copy Markdown
Contributor Author

schlunma commented Jun 1, 2026

After adding a series to this diagnostic it seems that one of the regressions tests is not failing even thought the expected series is empty...

@lewisjared
Copy link
Copy Markdown
Contributor

Can you make an issue to capture that?

@schlunma
Copy link
Copy Markdown
Contributor Author

schlunma commented Jun 2, 2026

See #703.

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.

2 participants