Skip to content

Updates to Technote section 2.9 Stomatal Resistance and Photosynthesis#4053

Merged
slevis-lmwg merged 13 commits into
ESCOMP:b4b-devfrom
slevis-lmwg:section_2.9_stom_resis_psn
Jun 3, 2026
Merged

Updates to Technote section 2.9 Stomatal Resistance and Photosynthesis#4053
slevis-lmwg merged 13 commits into
ESCOMP:b4b-devfrom
slevis-lmwg:section_2.9_stom_resis_psn

Conversation

@slevis-lmwg

Copy link
Copy Markdown
Contributor

Description of changes

For link to the pdf with proposed changes see the corresponding issue:
#3855 (comment)

Specific notes

Contributors other than yourself, if any:
@gbonan

CTSM issues resolved or otherwise addressed, if any:
Resolves #3855

Requirements before merge:

@slevis-lmwg slevis-lmwg self-assigned this May 27, 2026
@slevis-lmwg slevis-lmwg added documentation additions or edits to user-facing documentation or its infrastructure b4b bit-for-bit docs:update Significant update or fix needed to existing documentation docs-loc:tech-note Relates to Technical Note (science) labels May 27, 2026
@slevis-lmwg slevis-lmwg moved this to In Progress in CLM documentation May 27, 2026
@slevis-lmwg slevis-lmwg moved this from Todo to In Progress in LMWG: Sprint Planning Board May 27, 2026
@slevis-lmwg slevis-lmwg linked an issue May 27, 2026 that may be closed by this pull request
8 tasks
@slevis-lmwg

slevis-lmwg commented May 27, 2026

Copy link
Copy Markdown
Contributor Author

@gbonan just a heads up that I will request your review of these updates when I'm done.
So far I have addressed 2a and 2b from your pdf.

@slevis-lmwg

Copy link
Copy Markdown
Contributor Author

@gbonan I have now addressed issue (3) from your pdf.

Regarding issue (4), I found the equation in this form:
jmax25top = ((2.59_r8 - 0.035_r8*min(max((t10(p)-tfrz),11._r8),35._r8)) * vcmax25top) * params_inst%jmax25top_sf
Does the jmax25top_sf (scaling factor) parameter at the end correspond to what's discussed in the Canopy Scaling subsection of Section 2.9? That I'm not sure about. Do you know?

@slevis-lmwg

Copy link
Copy Markdown
Contributor Author

Answered my question about the missing jmax25top_sf in the documentation:
jmax25top_sf = 1 in the clm paramfile, so it's ok to not mention in the documentation.

I have now addressed issue (6) and I will go along with your assessment in issue (7) rather than pursuing further.

@slevis-lmwg

Copy link
Copy Markdown
Contributor Author

I left the most difficult (or time consuming) of Gordon's issues for last, and I am considering opening a separate PR for it for clearer tracking. This is what's left:

Parameters are given in Table 2.9.1, Table 2.9.2, or are in the text. These need to be updated. There are a lot!

@slevis-lmwg

slevis-lmwg commented May 30, 2026

Copy link
Copy Markdown
Contributor Author

Notes to self:

  • Paused at Rd25 before eq. 2.9.9 (check params in SP mode; BGC mode already addressed)
  • Saved latest paramfile in /glade/derecho/scratch/slevis/ctsm60_params.c260518.asc`

@wwieder

wwieder commented Jun 1, 2026

Copy link
Copy Markdown
Contributor

Are you ready for a review on this one, @slevis-lmwg, or still working?

@slevis-lmwg

Copy link
Copy Markdown
Contributor Author

Still working through the parameters.

@slevis-lmwg

Copy link
Copy Markdown
Contributor Author

Ready for review. Reaching out to @gbonan but others are welcome to look, as well. Gordon, I will also reach out to you via email or chat.

@slevis-lmwg slevis-lmwg moved this from In Progress to In review in CLM documentation Jun 2, 2026
@slevis-lmwg slevis-lmwg moved this from In Progress to Stalled in LMWG: Sprint Planning Board Jun 2, 2026
Comment thread doc/source/tech_note/Photosynthesis/CLM50_Tech_Note_Photosynthesis.rst Outdated
Change gamma to gamma star
Comment thread doc/source/tech_note/Photosynthesis/CLM50_Tech_Note_Photosynthesis.rst Outdated
Change gamma to gamma star
@slevis-lmwg slevis-lmwg requested a review from samsrabin June 3, 2026 21:41

@samsrabin samsrabin left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Stamp of approval for Gordon.

@slevis-lmwg slevis-lmwg merged commit c1024c1 into ESCOMP:b4b-dev Jun 3, 2026
7 checks passed
@github-project-automation github-project-automation Bot moved this from In review to Done in CLM documentation Jun 3, 2026
@github-project-automation github-project-automation Bot moved this from Stalled to Done in LMWG: Sprint Planning Board Jun 3, 2026
@slevis-lmwg slevis-lmwg deleted the section_2.9_stom_resis_psn branch June 3, 2026 23:41
@ekluzek ekluzek mentioned this pull request Jun 8, 2026
10 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

b4b bit-for-bit docs:update Significant update or fix needed to existing documentation docs-loc:tech-note Relates to Technical Note (science) documentation additions or edits to user-facing documentation or its infrastructure

Projects

Status: Done
Status: Done

Development

Successfully merging this pull request may close these issues.

Review 2.9. Stomatal Resistance and Photosynthesis

3 participants