cedadev / ccmi-2022

CCMI-2022 - in support of the WMO/UNEP Scientific Assessment of Ozone Depletion Report 2022
BSD 2-Clause "Simplified" License
0 stars 4 forks source link

mmarchand003 ceda-cc report for IPSL-CM6A-ATM-LR-REPROBUS refD2 data #67

Open mollymacrae opened 1 year ago

mollymacrae commented 1 year ago

The following ceda-cc metadata errors were raised for this upload:

mollymacrae commented 1 year ago

Variable clchl is not listed in the AmonZ table. Should this variable be added to the mip table?

mollymacrae commented 1 year ago

frequency attribute errors

for variables in tables:

10AdayPt: frequency attribute needs to be changed from '10day' to '10dayPt' to match the CV.json table.

Aday: frequency attribute needs to be changed from 'day' to 'dayPt' for all variables expect tasmax and tasmin to match the CV.json table.

mollymacrae commented 1 year ago

incorrect bounds attribute for time_bnds variable: The following error is raised for all variables with 'time' coordinate: ERROR C4.003.001: [time_attributes]: FAILED:: Required bounds attribute not present or not correct value

I can see that there is a bounds attribute in the files with value 'time:bounds = "time_bounds"'.

Comparing with other ccmi2022 data, they have a variable 'time_bnds' and corresponding time attribute 'time:bounds = "timebnds"' (whereas for this upload it is called 'timebounds'). I believe this is why the checks are raising this error.

This is the following files:

LolaFalletti commented 1 year ago

Variable clchl is not listed in the AmonZ table. Should this variable be added to the mip table?

Hello, This is not a new variable. During the simulation we had to use the name clchl instead of cl because cl is already used for something else in CMIP6 (and our post-processing tool is based on CMIP6), and I forgot to change the name afterwards. I will change the name to cl.

LolaFalletti commented 1 year ago

frequency attribute errors

for variables in tables:

10AdayPt: frequency attribute needs to be changed from '10day' to '10dayPt' to match the CV.json table.

Aday: frequency attribute needs to be changed from 'day' to 'dayPt' for all variables expect tasmax and tasmin to match the CV.json table.

Ok, I will change this.

charliepascoe commented 1 year ago

Thanks Lola. Molly is away for the next couple of weeks, I'm going to try and keep up with the CCMI uploads in the mean time. Charlotte

LolaFalletti commented 1 year ago

incorrect bounds attribute for time_bnds variable: The following error is raised for all variables with 'time' coordinate: ERROR C4.003.001: [time_attributes]: FAILED:: Required bounds attribute not present or not correct value

I can see that there is a bounds attribute in the files with value 'time:bounds = "time_bounds"'.

Comparing with other ccmi2022 data, they have a variable 'time_bnds' and corresponding time attribute 'time:bounds = "timebnds"' (whereas for this upload it is called 'timebounds'). I believe this is why the checks are raising this error.

This is the following files:

* all Amon files

* all AmonZ files

* variables tasmax and tasmin

I saw that time:bounds = "time_bounds" ; is present in other files (Aday etc) too, do I need to change it everywhere, or only in Amon, AmonZ files and variables tasmax and tasmin ?

LolaFalletti commented 1 year ago

Thanks Lola. Molly is away for the next couple of weeks, I'm going to try and keep up with the CCMI uploads in the mean time. Charlotte

Hello Charlotte, Ok, we will let you know when Marion has uploaded the new files (probably in the next few days).

LolaFalletti commented 8 months ago

Hello, I would like to know if you have managed to ingest these data to the CEDA archive? I'm sorry, I realize that I didn't let you know when Marion had uploaded the news files, but it was done just after my last message. Thanks a lot in advance, Lola