WCRP-CMIP / CMIP6_CVs

Controlled Vocabularies (CVs) for use in CMIP6
Creative Commons Attribution 4.0 International
152 stars 78 forks source link

Augment activity_id to provide description with identifier #397

Closed durack1 closed 6 years ago

durack1 commented 6 years ago

Augment activity_id to be self-describing, following full descriptions at https://www.wcrp-climate.org/modelling-wgcm-mip-catalogue/modelling-wgcm-cmip6-endorsed-mips e.g.

{
   "activity_id":{
      "AerChemMIP":"Aerosols and Chemistry Model Intercomparison Project",
      "C4MIP":"Coupled Climate Carbon Cycle Model Intercomparison Project",
      "CFMIP":"Cloud Feedback Model Intercomparison Project",
      "CMIP":"CMIP DECK: 1pctCO2, abrupt4xCO2, amip, esm-piControl, esm-historical, historical, and piControl experiments",
...
}
durack1 commented 6 years ago

@momipsl, @markelkington, @martinjuckes, @MartinaSt, @ARadhakrishnanAtGFDL will it impact any of you if we make the change noted above, and similar changes captured in #396 and #395?

We have noted that the identifiers are not self-describing, and converting this information to a dictionary, with both key and description provides more context

@taylor13

MartinaSt commented 6 years ago

@durack1 Thanks for asking: source_type and grid_label are no problem for the citation. In case of activity_id there might be a minor change in the insert script required but this is no problem.

markelkington commented 6 years ago

Paul

No problem for me – I think it helps to have a formal description

Mark

From: Paul J. Durack [mailto:notifications@github.com] Sent: 31 August 2017 18:55 To: WCRP-CMIP/CMIP6_CVs Cc: Elkington, Mark; Mention Subject: Re: [WCRP-CMIP/CMIP6_CVs] Augment activity_id to provide description with identifier (#397)

@momipslhttps://github.com/momipsl, @markelkingtonhttps://github.com/markelkington, @martinjuckeshttps://github.com/martinjuckes, @MartinaSthttps://github.com/martinast, @ARadhakrishnanAtGFDLhttps://github.com/aradhakrishnanatgfdl will it impact any of you if we make the change noted above, and similar changes captured in #396https://github.com/WCRP-CMIP/CMIP6_CVs/issues/396 and #395https://github.com/WCRP-CMIP/CMIP6_CVs/issues/395 ?

We have noted that the identifiers are not self-describing, and converting this information to a dictionary, with both key and description provides more context

@taylor13https://github.com/taylor13

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/WCRP-CMIP/CMIP6_CVs/issues/397#issuecomment-326373816, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AFTIJ9xnI50scE3SUO9jVyisqUApJ3Oaks5sdvNmgaJpZM4PIHuu.