-
Notifications
You must be signed in to change notification settings - Fork 139
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 iceruf
to namelist
#600
Conversation
Allow setting the ice surface roughness at the atmosphere interface, 'iceruf', in the CICE namelist. This makes it easier to modify this value.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since iceruf is an icepack parameter, it will need to be added to the interface routines (init, query, etc). I suspect that if you tried to change its value in this version, you'd still get BFB with the unmodified code!
In fact it is already present in the Icepack interface, so I just had to wire it up in CICE. I'm passing CICE/cicecore/cicedynB/general/ice_init.F90 Lines 1788 to 1805 in 5d71476
(see the diff). I think that's all that's needed, I did do a manual test by changing the iceruf value in the namelist. |
Ah yes, sorry for missing that. Looks good, thanks. |
PR checklist
Short (1 sentence) summary of your PR:
Allow setting the ice surface roughness at the atmosphere interface,
iceruf
, in the CICE namelist. This makes it easier to modify this value.Developer(s):
Original-pach-by: probably F. Roy or F. Dupont, ECCC
Suggest PR reviewers from list in the column to the right.
Please copy the PR test results link or provide a summary of testing completed below.
This is a another small step in bringing our in-house modifications back to the Consortium. More to come in the coming weeks / months.