ctsm5.4.040: Update clm6 paramfiles and add a new one for hillslope hydrology#4029
Merged
Conversation
slevis-lmwg
commented
May 18, 2026
Contributor
Author
|
Ok, now ready for review, and we can first discuss if you prefer @wwieder. |
ekluzek
approved these changes
May 20, 2026
Contributor
ekluzek
left a comment
There was a problem hiding this comment.
The basic functionality is what needs to be done.
I'm hitting approve so you don't have to track me down. But, the one suggested change I gjive is something that I think should be done. And I think it's straightforward enough to do. Otherwise, we can do it later.
And there's a more subtle thing that came to my mind that will need more thought and some discussion to figure out, so should be done later.
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
Updating the clm6 paramfiles.
Specific notes
Contributors other than yourself, if any:
@wwieder @linniahawkins
CTSM issues resolved or otherwise addressed, if any:
Resolves #3659
If answers are expected to change, describe (delete this line otherwise):
Answers change from the baseline due to changing param values in the CLM6 paramfiles.
Any user interface changes (namelist or namelist defaults changes)?
Changing two paramfiles and adding a new one for hillslope hydrology in namelist_defaults.
Testing planned or performed, if any:
./run_sys_tests -s aux_clm -c ctsm5.4.039 -g ctsm5.4.040./run_sys_tests -s aux_clm -c ctsm5.4.039 -g ctsm5.4.040Requirements before merge: