Decomposition tech note updated for CLM6 - #4063
Merged
Merged
Conversation
wwieder
self-requested a review
May 28, 2026 22:09
8 tasks
This comment was marked as resolved.
This comment was marked as resolved.
Member
|
The build errors should be resolved now. |
Contributor
|
@slevis-lmwg I'm passing the review of this PR off to you. Thanks in advance. |
slevis-lmwg
reviewed
Jul 18, 2026
slevis-lmwg
left a comment
Contributor
There was a problem hiding this comment.
@katierocci I made some suggestions for edits based on the updates that I saw in your branch here in the PR. I will come back to your PR again next week to look more carefully at the "built" version of this chapter.
Thank you for all the work that you put into this!
…on.rst Replace hardwired figure number with reference
…on.rst Replace reference and hardwired table number with links
…on.rst Replace hardwired reference with link
…on.rst Replace hardwired chapter number and 2 references with links
slevis-lmwg
requested changes
Jul 21, 2026
slevis-lmwg
left a comment
Contributor
There was a problem hiding this comment.
@katierocci here are a few more changes that I was hoping you could confirm before I make them.
This comment was marked as resolved.
This comment was marked as resolved.
slevis-lmwg
reviewed
Aug 6, 2026
Fix typo in equation references
slevis-lmwg
approved these changes
Aug 6, 2026
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
The Decomposition page of the Tech Note has been updated for CLM6 to remove text about the old soil submodel and text has been added for the MIMICS soil submodel. The equations have been updated accordingly.
Specific notes
Contributors other than yourself, if any:
CTSM issues resolved or otherwise addressed, if any:
Requirements before merge: