Skip to content

Update phenology tech note#3960

Merged
slevis-lmwg merged 10 commits into
ESCOMP:b4b-devfrom
linniahawkins:edit-docs
Apr 30, 2026
Merged

Update phenology tech note#3960
slevis-lmwg merged 10 commits into
ESCOMP:b4b-devfrom
linniahawkins:edit-docs

Conversation

@linniahawkins
Copy link
Copy Markdown
Contributor

Description of changes

Added Leah Birch 2021 updates to boreal and arctic seasonal deciduous phenology onset and offset triggers.

Fixed some errors and revised phenology documentation for clarity.

Specific notes

Contributors other than yourself, if any:

CTSM Issues Fixed (include github issue #): #3867

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

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

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

Testing performed, if any: Checked rendering, looked good.

@samsrabin
Copy link
Copy Markdown
Member

samsrabin commented Apr 24, 2026

@linniahawkins the git-fleximod test failure here is because the PR updates FATES but not the corresponding entry in .gitmodules. I'm assuming that's unintended, so to resolve, (1) make sure your git status has no uncommitted changes and then (2):

bin/git-fleximod update fates
git add fates
git commit -m "Revert accidental FATES update"

@linniahawkins
Copy link
Copy Markdown
Contributor Author

Shoot - okay I think I understand what happened. I'll try again.

@linniahawkins linniahawkins marked this pull request as draft April 24, 2026 20:22
@linniahawkins linniahawkins marked this pull request as ready for review April 24, 2026 20:54
@linniahawkins
Copy link
Copy Markdown
Contributor Author

Okay I think I was able to fix it, thanks

Copy link
Copy Markdown
Contributor

@slevis-lmwg slevis-lmwg left a comment

Choose a reason for hiding this comment

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

@linniahawkins thank you for your edits and updates in this section of the Technote. I have a few comments and a few requests for changes. I realize that I'm raising preexisting issues in some cases, but I think it's worthwhile fixing them.

@samsrabin samsrabin added documentation additions or edits to user-facing documentation or its infrastructure b4b bit-for-bit docs-loc:tech-note Relates to Technical Note (science) labels Apr 30, 2026
@samsrabin samsrabin changed the base branch from master to b4b-dev April 30, 2026 16:31
@samsrabin samsrabin moved this to In review in CLM documentation Apr 30, 2026
Copy link
Copy Markdown
Contributor

@slevis-lmwg slevis-lmwg left a comment

Choose a reason for hiding this comment

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

@linniahawkins thank you for your careful work in this PR. I had not realized how much extra work I was requesting with my review. Your PR could serve as an example for what we should all do to update our Sections.

I'm approving, and I will subsequently merge to b4b-dev.

…Note_Vegetation_Phenology_Turnover.rst


typo correction
@slevis-lmwg slevis-lmwg merged commit 7b970d5 into ESCOMP:b4b-dev Apr 30, 2026
6 checks passed
@github-project-automation github-project-automation Bot moved this from In review to Done in CLM documentation Apr 30, 2026
@slevis-lmwg slevis-lmwg linked an issue May 1, 2026 that may be closed by this pull request
8 tasks
@linniahawkins
Copy link
Copy Markdown
Contributor Author

Thanks @slevis-lmwg - Very excited to have my first PR merged!

@slevis-lmwg slevis-lmwg mentioned this pull request May 7, 2026
3 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-loc:tech-note Relates to Technical Note (science) documentation additions or edits to user-facing documentation or its infrastructure

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Review 2.21. Vegetation Phenology and Turnover

3 participants