Skip to content

ctsm5.4.035: Merge b4bdev 20260424#3969

Merged
ekluzek merged 82 commits intoESCOMP:masterfrom
ekluzek:merge-b4bdev-20260424
Apr 27, 2026
Merged

ctsm5.4.035: Merge b4bdev 20260424#3969
ekluzek merged 82 commits intoESCOMP:masterfrom
ekluzek:merge-b4bdev-20260424

Conversation

@ekluzek
Copy link
Copy Markdown
Contributor

@ekluzek ekluzek commented Apr 24, 2026

Description of changes

Update submodules, enforce irrigate to off for FATES transient cases, and several documentation updates. Including a github action that will trigger when a PR has a build failure for the documentation. It will then add a comment to the PR summarizing the error. Hopefully, this will be easier for doc editors to understand what broke in the build for their PR.

When a PR has the "build docs" tests fail, it's hard to figure out why. This PR will make it so that the doc build log is posted in a PR comment after the test failure. Example samsrabin#17 (comment).

Since ctsm5.3.046: patch%itype is set to -999, which produces out-of-bounds error, when running HIST with fates, since the defaults point to irrigate=.true..

Though there is irrigate=.true. in use_cases, all the statements with .true. value have use_crop=.true. which can not be true when fates is on.

PR's:

#3624
#3925
#3955
#3938
#3812

Specific notes

Contributors other than yourself, if any: @dmleung @nmizukami @samsrabin @olyson @mvdebolskiy Claude Sonnet 4.6 and Opus 4.7 1M

CTSM Issues Fixed (include github issue #):
Fixes #3170
Most of #3861 except #3968

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

Any User Interface Changes (namelist or namelist defaults changes)? Yes
Change so that irrigate will be false when running with FATES for transient cases

Does this create a need to change or add documentation? Did you do so? No But, there are documentation updates

Testing performed, if any: running regular

dmleung and others added 30 commits November 22, 2025 04:23
…nstants for the dust doc. Removed description of Zender 2003 dust emission scheme in the doc.
…20260409

Merge master ctsm5.4.031 to b4b-dev
Co-authored-by: Danny M. Leung <97262277+dmleung@users.noreply.github.com>
Co-authored-by: Danny M. Leung <97262277+dmleung@users.noreply.github.com>
Co-authored-by: Danny M. Leung <97262277+dmleung@users.noreply.github.com>
Co-authored-by: Danny M. Leung <97262277+dmleung@users.noreply.github.com>
Co-authored-by: Danny M. Leung <97262277+dmleung@users.noreply.github.com>
Co-authored-by: Danny M. Leung <97262277+dmleung@users.noreply.github.com>
Co-authored-by: Danny M. Leung <97262277+dmleung@users.noreply.github.com>
Co-authored-by: Danny M. Leung <97262277+dmleung@users.noreply.github.com>
Co-authored-by: Danny M. Leung <97262277+dmleung@users.noreply.github.com>
Co-authored-by: Danny M. Leung <97262277+dmleung@users.noreply.github.com>
Co-authored-by: Danny M. Leung <97262277+dmleung@users.noreply.github.com>
ekluzek and others added 6 commits April 24, 2026 10:22
…oute

Revision of section 2.15 (mizuRoute) in technical note for CLM6
New documentation for the CTSM dust emission module
bug fix to the FATES land use driver input code

This PR includes a cherry-picked fix from NorESMhub#209 to address swap in the
order of rangeland and pasture in the reading of the landuse drivers
@ekluzek ekluzek self-assigned this Apr 24, 2026
@ekluzek ekluzek added enhancement new capability or improved behavior of existing capability external issue needs to be addressed elsewhere (submodule); issue here for the sake of project tracking b4b bit-for-bit labels Apr 24, 2026
@ekluzek ekluzek added PR status: needs testing test: aux_clm Pass aux_clm suite before merging docs-loc:tech-note Relates to Technical Note (science) labels Apr 24, 2026
@github-project-automation github-project-automation Bot moved this to Ready to start (or start again) in CTSM: Upcoming tags Apr 24, 2026
Copy link
Copy Markdown
Member

@samsrabin samsrabin left a comment

Choose a reason for hiding this comment

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

Approving without review, assuming this is a straightforward merge.

@github-project-automation github-project-automation Bot moved this from Ready to start (or start again) to In progress - master in CTSM: Upcoming tags Apr 24, 2026
@ekluzek ekluzek changed the title Merge b4bdev 20260424 ctsm5.4.035: Merge b4bdev 20260424 Apr 24, 2026
@ekluzek ekluzek moved this from Todo to In Progress in LMWG: Sprint Planning Board Apr 24, 2026
@ekluzek
Copy link
Copy Markdown
Contributor Author

ekluzek commented Apr 25, 2026

There are four tests that are still in the queue on Derecho. But, otherwise all testing looks as expected on both Derecho and Izumi. I might bump the remaining tests to premium if they don't run pretty soon...

@ekluzek ekluzek merged commit 6c64ae6 into ESCOMP:master Apr 27, 2026
6 checks passed
@github-project-automation github-project-automation Bot moved this from In Progress to Done in LMWG: Sprint Planning Board Apr 27, 2026
@github-project-automation github-project-automation Bot moved this from In progress - master to Done (non release/external) in CTSM: Upcoming tags Apr 27, 2026
@ekluzek ekluzek deleted the merge-b4bdev-20260424 branch April 27, 2026 04:05
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) enhancement new capability or improved behavior of existing capability external issue needs to be addressed elsewhere (submodule); issue here for the sake of project tracking PR status: needs testing test: aux_clm Pass aux_clm suite before merging

Projects

Status: Done (non release/external)
Status: Done

Development

Successfully merging this pull request may close these issues.

Docs needed: Dust updates

6 participants