Rewrite how to run ACCESS-ESM1.5 according to feedback from user experience#1202
Rewrite how to run ACCESS-ESM1.5 according to feedback from user experience#1202ccarouge wants to merge 48 commits into
Conversation
d0d207f to
3daa37a
Compare
252fd18 to
b8d41c2
Compare
|
a788dae to
7f52ac0
Compare
02d22f8 to
a89c91c
Compare
b4c8230 to
c24f202
Compare
|
@paigem Here is (finally) a first version of ESM1.5 documentation with include bits and collapsible sections. I haven't read the whole page to make sure it is all coherent yet but I wanted to get your opinion on the balance between information directly on the page and in collapsible sections. I have tried to follow these principles:
I'll continue working on it, reading the page, fixing details and cleaning up the payu.md page that is used for the include parts but let me know if you think the balance of information looks alright. |
2c36319 to
3e369ec
Compare
anton-seaice
left a comment
There was a problem hiding this comment.
Looks great @ccarouge - this should make it simpler long term!
| When the model fails or completes a run, PBS writes the standard output and error streams to two files inside the _control_ directory: `<jobname>.o<job-ID>` and `<jobname>.e<job-ID>`, respectively. | ||
|
|
||
| These files usually contain logs about _payu_ tasks, and give an overview of the resources used by the job.<br> | ||
| To move these files to the _archive_ directory, use the following commmand: |
There was a problem hiding this comment.
| When the model fails or completes a run, PBS writes the standard output and error streams to two files inside the _control_ directory: `<jobname>.o<job-ID>` and `<jobname>.e<job-ID>`, respectively. | |
| These files usually contain logs about _payu_ tasks, and give an overview of the resources used by the job.<br> | |
| To move these files to the _archive_ directory, use the following commmand: | |
| When the model fails or completes a run, PBS writes the standard output and error streams to two files inside the _control_ directory: `<jobname>.o<job-ID>` and `<jobname>.e<job-ID>`, respectively. These files usually contain logs about _payu_ tasks, and give an overview of the resources used by the job.<br> | |
| To move these files to the _archive_ directory, use the following commmand: |
| As shown in the diagram, the general layout of a _payu_-supported model run consists of two main directories: | ||
|
|
||
| - The _control_ directory contains the model configuration and is the directory from which the model run is started. | ||
| This directory contains information to manage the simulation and the scientific options that define the algorithms used in the model component or the diagnostics saved by the model component. The simulation is orchestrated from a main `config.yaml` file contained in this directory. The files specific to each model component are either located directly in the _control_ directory if the model has one model component, or in subdirectories if the model has several model components. The `submodels` section of the `config.yaml` file specifies the name of the submodels and of the subdirectories containing the pertinent files. To modify these options please refer to the configurations documentation of the respective model component. |
There was a problem hiding this comment.
This paragraph is a bit long/verbose. Can we break it up or use dot points or something ?
| <terminal-line data="input">payu --version</terminal-line> | ||
| <terminal-line lineDelay="1000">payu 1.1.6</terminal-line> | ||
| </terminal-window> | ||
| ??? info "What is _payu_ and how are the simulation files organised by _payu_" |
There was a problem hiding this comment.
| ??? info "What is _payu_ and how are the simulation files organised by _payu_" | |
| ??? info "What is _payu_ and how are simulation files organised by _payu_" |
There was a problem hiding this comment.
Is this heading going in all pages? Could is go in payu.md instead ?
|
|
||
| All released {{ model }} configurations are available from the [{{ model }} configs]({{github_configs}}) GitHub repository.<br> | ||
| Released configurations are tested and supported by ACCESS-NRI, as an adaptation of those originally developed by [CSIRO](https://www.csiro.au/en/research/environmental-impacts/climate-change/climate-science-centre) and [CLEX CMS](https://github.com/coecms/access-esm). | ||
| Released configurations are tested and supported by ACCESS-NRI, as an adaptation of those originally developed by [CSIRO](https://www.csiro.au/en/research/environmental-impacts/climate-change/climate-science-centre) and [CLEX CMS](https://github.com/coecms/access-esm).<br> |
There was a problem hiding this comment.
| Released configurations are tested and supported by ACCESS-NRI, as an adaptation of those originally developed by [CSIRO](https://www.csiro.au/en/research/environmental-impacts/climate-change/climate-science-centre) and [CLEX CMS](https://github.com/coecms/access-esm).<br> | |
| Released configurations are tested and supported by ACCESS-NRI, and build upon those originally developed by [CSIRO](https://www.csiro.au/en/research/environmental-impacts/climate-change/climate-science-centre) and [CLEX CMS](https://github.com/coecms/access-esm).<br> |
| | CMIP6 Concentration-driven pre-industrial | release-preindustrial+concentrations | | ||
| | CMIP6 Concentration-driven historical | release-historical+concentrations |
There was a problem hiding this comment.
If we are talking about CMIP6, then use piControl and historical I think?
These are answer changing to the published CMIP6 data, so do we need some clarifying language about what we mean by CMIP6 here ?
There was a problem hiding this comment.
The change in answers comes from changes in the code or compilation, correct?
There was a problem hiding this comment.
I would guess there's also runtime changes which have been answer changing, but am not really involved in esm1.5 so i don't know
There was a problem hiding this comment.
I don't understand: "If we are talking about CMIP6, then use piControl and historical I think?" Do you mean instead of "concentration-driven pre-industrial" and "concentration-driven historical"? There were some emissions-driven runs done for CMIP6.
There was a problem hiding this comment.
Do you mean instead of "concentration-driven pre-industrial" and "concentration-driven historical"?
Yes - exactly, if we are referring to explicit experiments, then use the CMIP name for them
| ### Get the model configuration | ||
|
|
||
| <!--start:get-config-payu--> | ||
| To get a local copy of a configuration, you need to: |
There was a problem hiding this comment.
| To get a local copy of a configuration, you need to: | |
| Before downloading (cloning) a local copy of a configuration, you need to: |
| ACCESS-NRI developed the [Model Live Diagnostics](/model_evaluation/evaluation_on_gadi/model_live_diagnostics) framework to check, monitor, visualise, and evaluate model behaviour and progress of ACCESS models currently running on _Gadi_.<br> | ||
| For a complete documentation on how to use this framework, check the [Model Diagnostics documentation](https://med-live-diagnostics.readthedocs.io/en/latest/index.html). |
There was a problem hiding this comment.
Does this work? looks stale since 2024 ?
There was a problem hiding this comment.
I was wondering about that, since it was on ESM1.5 and OM2 pages, I didn't remove it. Happy to remove though.
There was a problem hiding this comment.
Feel free to argue it's not in scope (:
| To run the model for longer than the default run length, conduct multiple runs, see | ||
| [Run an experiment](#run-an-experiment). | ||
|
|
||
| #### Understand _runtime_, _runspersub_, and _-n_ parameters {: id="multiple-runs"} |
There was a problem hiding this comment.
It's confusing to say that we don't recommend changing the run length, but oh, actually, here's how to change the run length ??
| days: 0 | ||
| ``` | ||
|
|
||
| With the default configuration settings, the sea ice component of {{ model }} will produce restart files only at the end of each year. If valid restart files are required when running shorter simulations, the sea ice model configuration should be modified so that restart files are produced at monthly frequencies. To do this, change the `dumpfreq = 'y'` setting to `dumpfreq = 'm'` in the `cice_in.nml` configuration file located in the `ice` subdirectory of the _control_ directory. |
There was a problem hiding this comment.
Doesn't payu crash if you don't have a restart file from the end of the run ?
There was a problem hiding this comment.
For cice it hangs onto the restart from the start of the run. It doesn't crash immediately, but I guess it would cause problems in subsequent runs
There was a problem hiding this comment.
Why are you assuming that the instructions in the text would not produce a restart at the end of the run @anton-seaice ? Or are you asking for an additional warning about that?
There was a problem hiding this comment.
It says If valid restart files are required when running shorter simulations, allowing for us not to require them if we wanted
There was a problem hiding this comment.
Oh, I understood it as: if you just run a short simulation and don't care about continuation, then you don't care about the creation of restarts by your short simulation. I'll update the text.
| <!--end:payu-restart-prune--> | ||
|
|
||
| <!--start:payu-advance-options--> | ||
| ### _payu_ advance options |
There was a problem hiding this comment.
| ### _payu_ advance options | |
| ### _payu_ options for advanced users |
blimlim
left a comment
There was a problem hiding this comment.
Thanks @ccarouge for putting this together. I really like the new structure, and don't have many suggested changes for the overall way its organised.
I haven't done too in depth of a review at the moment, but have added a few small comments on things that stood out to me.
|
|
||
| Inside the _laboratory_ directory, there are two subdirectories of particular interest: | ||
|
|
||
| - _work_ → for temporary storage of files needed by the model while it runs. _payu_ creates and removes directories and files in this directory upon successful completion of runs. It is left untouched in case of error to facilitate the identification of the cause of the model failure |
There was a problem hiding this comment.
I'd read the current text as suggesting the work directory is created at the end of each run. Would something like this be a little more explicit
| - _work_ → for temporary storage of files needed by the model while it runs. _payu_ creates and removes directories and files in this directory upon successful completion of runs. It is left untouched in case of error to facilitate the identification of the cause of the model failure | |
| - _work_ → for temporary storage of files needed by the model while it runs. _payu_ creates this directory at the start of each run and removes it upon their successful completion. It is left untouched in case of error to facilitate the identification of the cause of the model failure |
| <!--end:get-config-payu--> | ||
|
|
||
| <!--start:payu-clone-example--> | ||
| For example, if you want to do a sensitivity experiment in {{model}} using the configuration {{config_example}}. You decide the following: |
There was a problem hiding this comment.
Just a very small thing. I think "sensitivity experiment" complicates the example, and it could be clearer omitting it:
"For example, if you want to clone the configuration {{config_example}} for {{model}}, ..."
| payu run -n <number-of-runs> | ||
|
|
||
| This will run the configuration `number-of-runs` consecutive times for the configured run length. This way, the *total experiment length* will be `run-length * number-of-runs`. The `run-length` (i.e. the duration of each individual run) is defined in the configuration settings and its specification is model-dependent. | ||
| For example, to run an experiment for a total of 50 years using a configuration with a 5-year _run length_, the `number-of-runs` should be set to `10`: |
There was a problem hiding this comment.
Related to @anton-seaice's comment here, since we strongly suggest leaving the run length as 1 year, it might be best to use a 1 year run length in this example.
| <!--start:payu-PBS-resources--> | ||
| ### Modify PBS resources | ||
|
|
||
| If the model has been altered and needs more time or memory to complete, or needs to be submitted under a different NCI project, you will need to modify the following options in the `config.yaml`: |
There was a problem hiding this comment.
| If the model has been altered and needs more time or memory to complete, or needs to be submitted under a different NCI project, you will need to modify the following options in the `config.yaml`: | |
| If the model has been altered and needs more time or memory to complete, you will need to modify the following options in the `config.yaml`: |
|
|
||
| This section tells _payu_ which driver to use for the main `model` configuration and the location of all `input` files that are common to all its model components. | ||
|
|
||
| The `name` field, for the model section, is not actually used for the configuration run, so it can be safely ignored. The `name` field is used for submodels (see below). |
There was a problem hiding this comment.
I think the top level name option selects the payu driver. @Qian-HuiChen is that correct?
| - **_MOSRS_ account**<br> | ||
| [MOSRS](https://code.metoffice.gov.uk) is a server run by the UKMO to support collaborative development with other partners organisations. MOSRS contains the source code and configurations for some model components in {{ model }} (e.g., the [UM](/models/model_components/atmosphere/#unified-model-um)).<br> | ||
| The [Met Office Science Repository Service (MOSRS)](https://code.metoffice.gov.uk) is a server run by the UK Met Office (UKMO) to support collaborative development with other partners organisations. MOSRS contains the source code and configurations for some model components in {{ model }} (e.g., the [UM](/models/model_components/atmosphere/#unified-model-um)).<br> | ||
| To apply for a _MOSRS_ account, please contact your [local institutional sponsor](https://opus.nci.org.au/display/DAE/Prerequisites). |
There was a problem hiding this comment.
Are there any changes that need to go here with the updated process? We could add an includes/ukmo.md file so that prerequisites can be shared across the UM based models?
There was a problem hiding this comment.
I have this open PR with updated MOSRS account instructions.
@ccarouge, the idea is to put these updated MOSRS instructions in an include file that can be used by "rose-cylc" and the "payu" instructions. I did a temporary fix in the ESM1.5 instructions to avoid confusion until the new ESM documentation is ready.
| %} | ||
|
|
||
| For example, if the required configuration is the co2 concentration driven pre-industrial configuration, then the branch to select is [`release-preindustrial+concentrations`](https://forum.access-hive.org.au/t/access-esm1-5-release-information/2352). | ||
| ??? abstract "Experiment example" |
There was a problem hiding this comment.
| ??? abstract "Experiment example" | |
| ??? abstract "Example: Cloning a configuration" |
I think the "experiment example" could be made a little clearer that it's about cloning configurations. Not sure that my suggestion is the best though
Clarify instructions for specifying a custom restart path during experiment setup.
f5a6a30 to
299e6bd
Compare


Motivation
Following user's feedback from user experience events, it appears that:
From us, we also have the requirement that we don't want to duplicate the same information on several pages, as it becomes difficult to manage and ensure all occurrences are kept up-to-date.
All these considerations have led the Hive Docs team to consider a reformatting of the Run a Model page. Here are the choices made:
Limitations / Difficulties coming from these choices:
Current status
This rewrite is using the
payu.mdpage that was being developed for ESM1.6. Thepayu.mdpage has not yet been clean up of extra text, this is on purpose in case anything there is useful during the review processNeed to review the TOC on the ESM1.5 page: we may want to remove some sections from the TOC.