Repository navigation
Specify q10 for HR and MR - #4229
Merged
Merged
Conversation
wwieder
commented
Sep 24, 2026
slevis-lmwg
approved these changes
Sep 24, 2026
Contributor
There was a problem hiding this comment.
Thanks @wwieder and @linniahawkins.
- Update last q10 to Q10hr
- Would you like me to run aux_clm to prepare for this PR's merge to b4b-dev?
Co-authored-by: Samuel Levis <slevis@ucar.edu>
Contributor
Author
Yes, please run testing and merge! Thanks |
Contributor
|
|
Contributor
|
Izumi finished successfully. UPDATE on derecho, I'm rerunning two tests that are known to eventually pass. |
12 tasks done
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.
Clarified which Q10 parameter is used for MR and HR.
Both are currently set to 1.5, but the q10_HR parameter is not used.
Specific notes
Contributors other than yourself, if any:
@linniahawkins found this bug
CTSM issues resolved or otherwise addressed, if any:
Resolves #4196
This should be b4b.
Note: this also needs to be corrected in FATES, @glemieux