Draft
Conversation
Contributor
|
@manishvenu do we want this for all DATM compsets, or just when we're running MOM6 in a regional configuration? |
Contributor
|
Also, this PR might not be necessary -- I just created a global |
Author
I'm not sure! Not entirely clear what it's for, so I think you may know more than me haha.
Huh, I'm not seeing any calls to ICE_NCAT in this tag but not entirely sure how it works. I'll continue to look |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Description of changes
ICE_NCAT is used by both CICE and MARBL (the ocean BGC model). We need the ICE_NCAT set to 1 so that data is passed to MARBL through the coupler (CICE buildnml always sets ICE_NCAT but DICE doesn't). It affects shortwave penetration. @mnlevy1981 can answer questions about this PR.
Specific notes
Contributors other than yourself, if any: @mnlevy1981
CDEPS Issues Fixed (include github issue #): N/A
Are there dependencies on other component PRs (if so list): N/A
Are changes expected to change answers (bfb, different to roundoff, more substantial):
Any User Interface Changes (namelist or namelist defaults changes):
ICE_NCAT changes from a default of zero to one.
Testing performed (e.g. aux_cdeps, CESM prealpha, etc): None
Hashes used for testing: