Conversation
As provided from F. Sevault; just an `ncdump -h`
jesusff
left a comment
There was a problem hiding this comment.
There are a few deviations from the DRS to be handled. See comments
| @@ -0,0 +1,90 @@ | |||
| netcdf tpoce_MED-12_ERA5_evaluation_r1i1p1f1_CNRM-MF_CNRM-RCSM6B-SN_v1-r1_mon_198803-198803 { | |||
There was a problem hiding this comment.
This is just an example for a single month, but the real monthly files should be split into decades, starting with year 1, that is, this month would belong to a file named ..._198101-199012.nc
| nav_lon:standard_name = "longitude" ; | ||
| nav_lon:long_name = "Longitude" ; | ||
| nav_lon:units = "degrees_east" ; | ||
| float deptht(deptht) ; |
There was a problem hiding this comment.
Should be depth(depth) (no t at the end) https://github.com/WCRP-CORDEX/cordex-cmip6-cmor-tables/blob/508ca0ee5df5c27663622371bbf34f0ba916970b/Tables/CORDEX-CMIP6_coordinate.json#L233-L254
| nav_lat:standard_name = "latitude" ; | ||
| nav_lat:long_name = "Latitude" ; | ||
| nav_lat:units = "degrees_north" ; | ||
| float nav_lon(y, x) ; |
There was a problem hiding this comment.
Should be lat and lon (no nav_)
The crs and native coordinates variables are missing
| nav_lon:units = "degrees_east" ; | ||
| float deptht(deptht) ; | ||
| deptht:name = "deptht" ; | ||
| deptht:long_name = "Vertical T levels" ; |
There was a problem hiding this comment.
should be long_name = "ocean depth coordinate"
| time:standard_name = "time" ; | ||
| time:long_name = "Time axis" ; | ||
| time:calendar = "gregorian" ; | ||
| time:units = "days since 1900-01-01 00:00:00" ; |
There was a problem hiding this comment.
should be days since 1950-01-01 unless the simulation started before 1950
| :references = "Nabat, P., Somot, S., Cassou, C., Mallet, M., Michou, M., Bouniol, D., Decharme, B., Druge, T., Roehrig, R. and Saint-Martin, D. (2020): Modulation of radiative aerosols effects by atmospheric circulation over the Euro-Mediterranean region, Atmos. Chem. Phys., 20, 8315-8349, doi:10.5194/acp-20-8315-2020." ; | ||
| :source = "CNRM-ALADIN64E1" ; | ||
| :source_id = "CNRM-ALADIN64E1" ; | ||
| :source_type = "AORCM AER" ; |
| :source = "CNRM-ALADIN64E1" ; | ||
| :source_id = "CNRM-ALADIN64E1" ; | ||
| :source_type = "AORCM AER" ; | ||
| :variable_id = "tpoce" ; |
| :project_id = "CORDEX" ; | ||
| :realm = "ocean" ; | ||
| :references = "Nabat, P., Somot, S., Cassou, C., Mallet, M., Michou, M., Bouniol, D., Decharme, B., Druge, T., Roehrig, R. and Saint-Martin, D. (2020): Modulation of radiative aerosols effects by atmospheric circulation over the Euro-Mediterranean region, Atmos. Chem. Phys., 20, 8315-8349, doi:10.5194/acp-20-8315-2020." ; | ||
| :source = "CNRM-ALADIN64E1" ; |
There was a problem hiding this comment.
source should be the content of the label_extended provided during registration. In general, all info registered for the source and institution should match that registered: https://wcrp-cordex.github.io/cordex-cmip6-cv/CORDEX-CMIP6_source_id.html
| :history = "none" ; | ||
| :institution = "CNRM (Centre National de Recherches Meteorologiques) - Météo France, Toulouse, France" ; | ||
| :institution_id = "CNRM-MF" ; | ||
| :license = "CORDEX model data produced by CNRM-MF is licensed under a Creative Commons Attribution 4.0 International License (https://creativecommons.org/licenses). Consult (...) for terms of use governing CORDEX output, including citation requirements and proper acknowledgment. The data producers and data providers make no warranty, either express or implied, including, but not limited to, warranties of merchantability and fitness for a particular purpose. All liabilities arising from the supply of the information (including any liability arising in negligence) are excluded to the fullest extent permitted by law." ; |
There was a problem hiding this comment.
license should be exactly https://cordex.org/data-access/cordex-cmip6-data/cordex-cmip6-terms-of-use
| :dr2xml_version = "3.0" ; | ||
| :domain = "MED-12" ; | ||
| :domain_id = "MED-12" ; | ||
| :driving_experiment = "ERA5, evaluation, r1i1p1f1" ; |
There was a problem hiding this comment.
driving_experiment is not free to choose, see https://github.com/WCRP-CORDEX/cordex-cmip6-cv/blob/main/CORDEX-CMIP6_driving_experiment_id.json
In general, all attributes in the CV must follow exactly the values there https://github.com/WCRP-CORDEX/cordex-cmip6-cv
This PR is to discuss the output data formatting using CDL files (
ndump -h file.nc > file.cdl). This is a text representation of netCDF file contents and shows the metadata and file structure. We can refine the example here and merge into main as an example when ready. Other examples can be added for variables from other components, or for models using different grids.In the files changed tab, you an comment on any line of the file. Anyone can edit the files in the PR branch to improve them.