Skip to content

Conversation

@rosiealice
Copy link
Collaborator

Description of changes

Altering defaults, to go along with NorESMhub/fates#48

Specific notes

Contributors other than yourself, if any:

CTSM Issues Fixed (include github issue #):

Are answers expected to change (and if so in what way)?

Any User Interface Changes (namelist or namelist defaults changes)?

Does this create a need to change or add documentation? Did you do so?

Testing performed, if any:
(List what testing you did to show your changes worked as expected)
(This can be manual testing or running of the different test suites)
(Documentation on system testing is here: https://github.com/ESCOMP/ctsm/wiki/System-Testing-Guide)
(aux_clm on derecho for intel/gnu and izumi for intel/gnu/nag/nvhpc is the standard for tags on master)

NOTE: Be sure to check your coding style against the standard
(https://github.com/ESCOMP/ctsm/wiki/CTSM-coding-guidelines) and review
the list of common problems to watch out for
(https://github.com/ESCOMP/CTSM/wiki/List-of-common-problems).

@rosiealice
Copy link
Collaborator Author

@mvdebolskiy given that this is needed for the fix to the h2o error, if I have new parameter updates (e.g. to address LAI biases) should I either

  1. append them to this PR or
  2. make a new PR?

@mvdebolskiy
Copy link
Collaborator

You should append both fates and these PR's. I will also add a fix for cplhist test and update CDEPS and share. And then go into ESCOMP updates in a tag after that.

@mvdebolskiy
Copy link
Collaborator

@rosiealice I assume I can fire off the tests?

@rosiealice
Copy link
Collaborator Author

Yes, it's good now.

@mvdebolskiy mvdebolskiy merged commit b1b0cee into NorESMhub:noresm Jan 30, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants