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

Add dump_last feature to icepack. #220

Closed
wants to merge 8 commits into from
Closed

Add dump_last feature to icepack. #220

wants to merge 8 commits into from

Conversation

dabail10
Copy link
Contributor

This adds the functionality to icepack to write a restart on the last step of a run. By default it is off. It will be turned on for smoke tests. Note, this is on top of the CMIP.

  • Developer(s): D. Bailey

  • Please suggest code Pull Request reviewers in the column at right.

  • Are the code changes bit for bit, different at roundoff level, or more substantial? BFB

  • Is the documentation being updated with this PR? (Y/N) N
    If not, does the documentation need to be updated separately at a later time? (Y/N) Y
    Note: "Documentation" includes information on the wiki and .rst files in doc/source/,
    which are used to create the online technical docs at https://readthedocs.org/projects/cice-consortium-icepack/.

  • Other Relevant Details:

Copy link
Contributor

@apcraig apcraig left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This PR has the CMIP changes in it. I think it should be closed until the CMIP changes are merged and the branch has been updated to the Consortium master. If the CMIP branch has further changes, the CMIP branch and this branch are going to get convoluted and we may lose some of the mods.

Also, I don't think set_nml.smoke is going to be picked up by the smoke test. There need to be some changes in icepack.setup to make that happen. At least that was my experience, but maybe I'm wrong. Have you confirmed that with these changes, the smoke test generates a restart at the end of the run?

@dabail10
Copy link
Contributor Author

I misunderstood the earlier conversation. I will close it for now.

@dabail10 dabail10 closed this Sep 11, 2018
lettie-roach pushed a commit to lettie-roach/Icepack that referenced this pull request Oct 18, 2022
…ortium#220)

* Added k1 basal stress parameter in namelist, added scripts for Philippe's machine

* Updated documentation for landfast parameterization
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants