Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PR checklist
Updates for RASM
apcraig
full test suite run on cheyenne, all tests pass as expected, https://github.com/CICE-Consortium/Test-Results/wiki/cice_by_hash_forks#2d31fffac6e7fdf11434758e4bb68befccbcefd6
Until we refactor the fixed bathymetry levels, we need the min(kmt,nlevel) otherwise the model will not run with kmt files used in coupled models (assuming they have more than 40 levels). Even with bathymetry and landice off, this code is executed and fails.
The other fix is associated with a renaming of the prognostic time manager variables names. This was done after the most recent RASM testing on the time manager branch, so that needed to be incorporated in the coupling layer.