-
Notifications
You must be signed in to change notification settings - Fork 224
docs: remove model-specific references from documentation #4976
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
Changes from all commits
9d7b6bf
ec76ad7
f25e523
96fd71e
a9b5842
2a8552d
eddaceb
334d6c8
1b10396
1ba4527
4aa7116
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -4,3 +4,4 @@ source/Tools_api | |
| source/CIME_api | ||
| source/_autosummary | ||
| source/generated | ||
| build/ | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -387,14 +387,14 @@ The steps for adding a new component grid to the model system follow. This proce | |
| At this time, if you are running with a new ocean or runoff grid, please contact Michael Levy (mlevy_AT_ucar_DOT_edu) for assistance. If you are running with standard ocean and runoff grids, the mapping file should already exist and you do not need to generate it. | ||
|
|
||
|
|
||
| 6. CESM specific: If you are adding a new atmosphere grid, this means you are also generating a new land grid, and you will need to create a new CLM surface dataset. (Otherwise you can skip this step). | ||
| You need to first generate mapping files for CLM surface dataset (since this is a non-standard grid). | ||
| 6. If you are adding a new atmosphere grid, this means you are also generating a new land grid, and you may need to create a new land surface dataset. (Otherwise you can skip this step). | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This section no longer applies to CESM. But in fact, more generally, this whole grids section needs to be rewritten since a lot of it no longer applies to CESM.
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I have this generalized in another PR. |
||
| You need to first generate mapping files for the land surface dataset (since this is a non-standard grid). | ||
| :: | ||
|
|
||
| > cd $CIMEROOT/../components/clm/tools/mkmapdata | ||
| > ./mkmapdata.sh --gridfile <lnd SCRIP grid file> --res <atm resolution name> --gridtype global | ||
|
|
||
| These mapping files are then used to generate CLM surface dataset. Below is an example for a current day surface dataset (model year 2000). | ||
| These mapping files are then used to generate the land surface dataset. Below is an example for a current day surface dataset (model year 2000). | ||
|
|
||
| :: | ||
|
|
||
|
|
@@ -403,7 +403,7 @@ The steps for adding a new component grid to the model system follow. This proce | |
|
|
||
| 7. Create grid file needed for create_newcase. | ||
| The next step is to add the necessary new entries in the appropriate ``config_grids.xml`` file. | ||
| You will need to modify ``$CIMEROOT/config/cesm/config_grids.xml`` or ``$CIMEROOT/config/e3sm/config_grids.xml`` depending on the value of ``$CIME_MODEL``. | ||
| You will need to modify ``$CIMEROOT/config/$model/config_grids.xml`` depending on the value of ``$CIME_MODEL``. | ||
| You will need to: | ||
|
|
||
| - add a single ``<model_grid>`` entry | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I feel like the removal of CESM here is misleading, since a lot of what's in this section is truly CESM (and maybe NorESM) specific. This probably needs to go in a new section that has a note that it's model-specific and still explicitly mentions CESM and maybe NorESM.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added a note.