Skip to content
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

Document CICE scripts #77

Closed
eclare108213 opened this issue Feb 7, 2018 · 5 comments
Closed

Document CICE scripts #77

eclare108213 opened this issue Feb 7, 2018 · 5 comments
Assignees

Comments

@eclare108213
Copy link
Contributor

especially ICE_RUNLENGTH

@eclare108213
Copy link
Contributor Author

The CICE documentation still has create.case instead of cice.setup, with the old flags. That should be fixed soon!

@duvivier
Copy link
Contributor

Oops, I just changed that all. Will do a PR soon.

@duvivier
Copy link
Contributor

duvivier commented Mar 28, 2018

Just adding to this (as a list for my reference):

  • The quickstart, user's guide, and developer's guides all need to be thoroughly tested and updated with the correct documentation in the *rst files.

List from @eclare108213 as part of Pull Request #106 (may not be complete):

  • user_guide/ug_performance.rst: the test setup information should be put somewhere else. Is this duplicated in user_guide/ug_testing.rst? If so, just delete it here.
  • user_guide/ug_running.rst: this is very old info, from cice v5! Maybe this is where to put setup info that's currently in ug_performance? We need to keep the table of environment variables.
  • user_guide/ug_testing.rst: update the cice.setup flags and remove the version/date line at the top
  • user_guide/ug_troubleshooting.rst: this is v5 info — update to v6
  • Many of the new files contain text specific to Icepack, which needs to be changed for CICE. E.g.
developer_guide/dg_about.rst, dg_driver.rst, dg_scripts.rst
  • remove all references to drupal (user_guide/ oceans11.lanl.gov/drupal)

@eclare108213
Copy link
Contributor Author

If ICE_RUNLENGTH has been documented, this issue can be closed. I don't remember seeing that. ??

@apcraig
Copy link
Contributor

apcraig commented Aug 13, 2018

ICE_RUNLENGTH is documented and I will close this, but we will need to review documentation before every release.

@apcraig apcraig closed this as completed Aug 13, 2018
JFLemieux73 pushed a commit to JFLemieux73/CICE that referenced this issue Mar 17, 2022
apcraig pushed a commit that referenced this issue Aug 16, 2024
…e cpl_scalar field when created for UFS (#969)


These are two commits cherry-picked from as in UFS and needed to close NOAA-EMC#84.

This PR adds the ability for CICE to write restart files at the end of the run (independent of other settings) and controlled via the CMEPS configuration option write_restart_at_endofrun. Setting this configuration option to True creates a restart file in the same way for CMEPS, MOM6, and CICE.

This PR also initializes the scalar field value for the index for NTile (implemented for FV3) not used or set in CICE. In certain cases, the scalar field value for this index has been found to be non-zero (NaN in debug compiles). This is the cause of the failure reported in ufs-community/ufs-weather-model#2338.

* Add end of run functionality to CICE (#77)
* initialize cpl_scalar field when created (#83)

---------

Co-authored-by: Daniel Sarmiento <42810219+dpsarmie@users.noreply.github.com>
Co-authored-by: Denise Worthen <denise.worthen@noaa.gov>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants