Fix fire diagnostic CMIP7 data selection and recipe writing#540
Fix fire diagnostic CMIP7 data selection and recipe writing#540lewisjared merged 1 commit intoClimate-REF:mainfrom
Conversation
adf1185 to
de33fa1
Compare
Codecov Report❌ Patch coverage is
Flags with carried forward coverage won't be shown. Click here to find out more.
🚀 New features to boost your workflow:
|
de33fa1 to
cd33274
Compare
|
To test this, run |
|
The CMIP6 test case works, but for me I'm getting the following error with CMIP7. I deleted any existing converted CMIP7 datasets and tried with #547 getting the same error. Was there a newer commit of ESMValCore that was meant to be used? |
|
Yes, the ESMValCore pull request in the top post. And there is also the open issue on ESMValTool. |
lewisjared
left a comment
There was a problem hiding this comment.
Ahh I see after rereading the description, the CMIP7 case is expected to fail still. The selected files are an improvement though.
I updated the pinned CMP7 readiness issue to link to ESMValGroup/ESMValTool#4343 for this diagnostic
Description
Fix fire diagnostic CMIP7 data selection and recipe writing. Unfortunately, the CMIP7 test case is now failing on ESMValGroup/ESMValCore#2981 and ESMValGroup/ESMValTool#4343.
Checklist
Please confirm that this pull request has done the following:
changelog/