Update CMIP6 CMOR tables to version 01.00.31 (6.8.31)

[bouweandela]

Tasks

• Create an issue to discuss what you are going to do, if you haven't done so already (and add the link at the bottom)
• This pull request has a descriptive title that can be used in a changelog
• Circle/CI tests pass. Status can be seen below your pull request. If the tests are failing, click the link to find out why.

If you need help with any of the tasks above, please do not hesitate to ask by commenting in the issue or pull request.

---

Related to #329, not sure if it solves that completely though.

Closes #364

[valeriupredoi]

can anyone check this is not in any of the issues on the pcmdi github page pls? @bouweandela @zklaus @jvegasbsc

[valeriupredoi]

https://github.com/PCMDI/cmor/issues/559

[valeriupredoi]

@bouweandela any reason why you pulled 6.8.31 and not the stable version 01.00.31?

[valeriupredoi]

not so fast @mattiarighi there are two standing issues: the actual version itself (see my comment above) and the issue with olevhalf that is currently looked into by the PCMDI folk 🍺

[bouweandela]

@bouweandela any reason why you pulled 6.8.31 and not the stable version 01.00.31?

6.8.31 is the latest release of the CMIP6 CMOR tables, see: https://github.com/PCMDI/cmip6-cmor-tables/releases. It contains version 01.00.31 of the CMIP6 data request.

[mattiarighi]

It would be good to have a quick go at one of the CMIP6 recipes with this new version @schlunma @debe-kevin

[valeriupredoi]

Cool! cheers @bouweandela 🍺 I reckon we should fix that Oclim table since the PCMDI guys have confirmed the bug (see issue I opened on their github page) Dr Valeriu Predoi. Computational scientist NCAS-CMS University of Reading Department of Meteorology Reading RG6 6BB United Kingdom

…

On Fri, 8 Nov 2019, 08:57 Bouwe Andela, ***@***.***> wrote: @bouweandela <https://github.com/bouweandela> any reason why you pulled 6.8.31 and not the stable version 01.00.31? 6.8.31 is the latest release of the CMIP6 CMOR tables, see: https://github.com/PCMDI/cmip6-cmor-tables/releases. It contains version 01.00.31 of the CMIP6 data request. — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#363?email_source=notifications&email_token=AG5EFI2ZEHM6G2NQZ22WOZDQSUSW3A5CNFSM4JKGDAE2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDPFXVA#issuecomment-551443412>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AG5EFI2EGJUD533SNEJEGTLQSUSW3ANCNFSM4JKGDAEQ> .

[valeriupredoi]

@schlunma Dr Valeriu Predoi. Computational scientist NCAS-CMS University of Reading Department of Meteorology Reading RG6 6BB United Kingdom

…

On Fri, 8 Nov 2019, 08:58 Mattia Righi, ***@***.***> wrote: It would be good to have a quick go at one of the CMIP6 recipes with this new version @schluma @debe-kevin <https://github.com/debe-kevin> — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#363?email_source=notifications&email_token=AG5EFI37KIES2PRPYV4BHDLQSUS4PA5CNFSM4JKGDAE2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDPF3SA#issuecomment-551443912>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AG5EFI6NWDDLA6EQ6XUD3D3QSUS4PANCNFSM4JKGDAEQ> .

[debe-kevin]

It would be good to have a quick go at one of the CMIP6 recipes with this new version @schlunma @debe-kevin

Hello @mattiarighi,
I tested the PR it on a rather simple recipe (consecdrydays, see below txt of the recipe).

recipe_consecdrydays_cmip6.txt

No error to report. I may run it on a more tricky recipe and let you know if I spot an error !
However, is there a specific dataset you want me to include in the recipe to test the PR ? @mattiarighi

[mattiarighi]

If you have the cmip6 version of recipe_perfmetrics at hand it would be good to try it.
Thank you!

[bouweandela]

@debe-kevin Any news?

[debe-kevin]

@debe-kevin Any news?

Hello @bouweandela ,
I ran the perfmetrics recipe on CMIP6 below. The recipe requires some fix files that I will push soon. Everything worked fine after adding the updated CMIP6 CMOR table :) !

recipe_perfmetrics_cmip6.txt

[bouweandela]

Thanks for testing! That is good news. @mattiarighi can this be merged then?

[mattiarighi]

The recipe requires some fix files that I will push soon.

Are these fixes required because of the updated CMOR tables?
If this is the case we should merge the fix files first, otherwise this PR would prevent some recipes to run.

[debe-kevin]

The recipe requires some fix files that I will push soon.

Are these fixes required because of the updated CMOR tables?
If this is the case we should merge the fix files first, otherwise this PR would prevent some recipes to run.

@mattiarighi Those are fixes for missing coordinates that I needed in order to run the recipe even before including the PR commits.

[bouweandela]

Yes, I would hope this pull request will reduce the number of fixes needed, since it removes some erroneously introduced auxiliary coordinates on fx files, see e.g. #364.