-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Experiment form review #6
Comments
Thanks for this solid first draft. Here are some suggestions: For “Experiment ID”, we should instruct those registering experiments to consult the full guidance document. I would replace the current text with: “There are rules for constructing acceptable experiment names which can be found in this guidance document. For “Description”, I suggest adding a sentence to the instruction: I think we should add an entry at this point in the form for “Experiment Title”, which we could describe as “A short phrase that can help in interpreting the unique experiment_id’s. As examples, the CMIP6 titles are listed here in the third column labeled “experiments”. For the registered “MIP / Activity ID” list, we should capitalize certain letters consistent with CMIP6: AerChemMIP, C4MIP, CFMIP, DAMIP, DCPP, GeoMIP, PMIP, RFMIP, ScenarioMIP. Also, shouldn’t “imip” be “ISMIP”? I don’t think the “Parent Activity ID” should be requested. Doesn’t the parent experiment ID imply a unique parent activity ID? If we retain this entry, we should indicate that “no parent” should be entered if an experiment is not initialized from another CMIP7 experiment. For “Allowed Model Component”, replace “must be included” with “may be included, but are not required”. Also we should note: If only the required model components should be included, enter “none”. For “Parent Experiment ID”, I think it is confusing to include “forcings” in the instructions. The parent of the historical run for example is the piControl, but most of the forcings are different in the historical period. I would simply eliminate “forcings” in the description. And I would mention that “no parent” is an acceptable entry. The text could read: The ID of the experiment providing the initial conditions. If not initialized from another CMIP7 experiment, enter “no parent”. In the case of “Sub-Experiment”, sometime ago I suggested that we eliminate this dataset descriptor in CMIP7. It was used only for DCPP runs in CMIP6. I propose that instead of defining a separate “sub-experiment”, we include it as part of the “ripf” variant identifier. We would replace the previously defined “initialization method” index (e.g., “i1”, “i2”, …) with the first full forecast year (e.g., “i1960”, “i1965”, ….). If alternative initialization methods were used, then a single letter suffix would be appended to the year (e.g., “i1960”, “i1960a”, “i1960b” …). Most modeling groups only use a single “method”, so it will be uncommon to see a letter suffix. Assuming this is acceptable (we need to check with DCPP coordinators), then we could eliminate “sub-experiment” on this form and ask that if there are multiple start dates for experiments, they should all be listed under “Start Date”, which could be renamed “Start Date(s)”. If we leave in “Sub-Experiment”, it should be renamed “Sub-Experiments”. We should provide more guidance for the “Tier” entry. Something like: “How essential is the experiment to the primary scientific goals of the activity? 1=absolutely essential, 2=of major value, 3=of lesser value.” I would rearrange the date/year input and provide further guidance as follows:
Note that there is no reason to request an “End Date” since that can be calculated from “Start Date” and “Number of Years”. |
Feedback from Martin: |
Done, but that document should be confirmed and added to the IPO webpages.
Done
Added above the description
Done
Done
I'd like to see something more concrete for a proposal for this before making this change. In particular I'd like to confirm that CMOR can handle the proposed changes.
Done
Done
Done
|
This needs a little more thought as to how to address. Will come back to this one.
I've added a small clarification.
I've kept it and expanded the description as noted above.
As suggested above I've removed the end date field in favour of the minimum number of years |
I've tried to adjust the wording to make this more consistent. Am happy to take further suggestions for changes |
Looking good. Perhaps a few additional tweaks are needed: At the top of the form it says "Add a title", and then "Add/Modify: Experiment: ". If the "item" is the name of the experiment, then this should read "new experiment_id", or something along those lines. Otherwise, what does "item" refer to? It's a bit confusing that for "activity id" and "parent experiment id", only 1 of the 2 options in each case is "required". Shouldn't both have an asterisk or neither? Perhaps all items should be required, but an acceptable answer for some is "NA" (not applicable), or something similar. Under "Source type codes for additional allowed model components", we say "Components that may be included in models when running this experiment." This sentence could be misinterpreted as asking for all components that can be included (including required components). Could we say instead "Components that may be included in models in addition to the required components listed above." I think those filling out the form would really appreciate it if we put the most common "parent experiments" first in the list. I would put: no parent, piControl, historical, amip, esm-piControl, esm-hist, followed by all the others. Also please use proper capitialization in all experiment_ids. (See CMIP6 listing here: https://wcrp-cmip.github.io/CMIP6_CVs/docs/CMIP6_experiment_id.html . Since "Start Date" can be left blank, its confusing to include the asterisk, indicating that it is required input. Perhaps the directions should say: If there is no parent experiment please enter "NA", or some other similar text. |
Experiment registration form
The new/modify experiment form has been added at
https://github.com/WCRP-CMIP/CMIP7_CVs/issues/new/choose
This issue is to capture feedback for any updates to this form
The text was updated successfully, but these errors were encountered: