-
Notifications
You must be signed in to change notification settings - Fork 301
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
Update submodule tags to pass runoff from CISM to ROF as per issue #2590 #2605
base: master
Are you sure you want to change the base?
Conversation
From the TODO list in the issue UPDATE: Trying aux_clm on izumi |
Bisecting between my passing and failing aux_clm test-suites:
|
aux_clm on izumi now OK on derecho
In the first of these two, the largest NORMALIZED DIFF is ctsm_sci tests_0703-123641de
In the last of these, I see larger diffs including in variables that I would not have expected initially. This is the only Bgc case here. Should I expect changes in CH4 variables as a result of the changes in river flow @swensosc?
|
@swensosc verified by email that these diffs seem reasonable. Sean wrote: |
Description of changes
Details in #2590
Specific notes
Contributors other than yourself, if any:
@ekluzek @mvertens @jedwards4b @billsacks
CTSM Issues Fixed (include github issue #):
Fixes #2590
Are answers expected to change (and if so in what way)?
Yes, due to change in default behavior
Any User Interface Changes (namelist or namelist defaults changes)?
Default behavior has changed as explained in #2590
Does this create a need to change or add documentation? Did you do so?
Probably, but I did not.