search

PCMDI / mip-cmor-tables

JSON tables for CMOR3 to create Model Intercomparison Project (MIP) datasets
Creative Commons Attribution 4.0 International
2 stars 4 forks source link

#42: Correct OPmon/tos validation limits #43

Closed matthew-mizielinski closed 7 months ago

matthew-mizielinski commented 7 months ago

Issue must have been a mismatch between the validation limits when I brought these in from the data request.

I've checked all tos entries across tables and none of the other tables have had the limits applied.

As such the only change is to the OPmon table.

Need to ensure that versioning is appropriately applied on release

matthew-mizielinski commented 7 months ago

We've agreed to remove all validation limits pending a review and clarification of their origin

taylor13 commented 7 months ago

I agree with this decision.

durack1 commented 7 months ago

Note to self there are 111 qc ranges in the CMIP6 data request (vs 2k variables)

matthew-mizielinski commented 7 months ago

Validation limits removed and table date updated in every file that has changed (i.e. not every single file).

@durack1 please confirm you are happy with this and I can then liaise with @wolfiex about a release tomorrow

matthew-mizielinski commented 7 months ago

Last two commits were to remove limits expressed in exponential form (this wasn't caught by the regex I had used to replace fields previously)