Skip to content

Factor out new subroutine CropPhase_OnePatch#3713

Merged
samsrabin merged 4 commits into
ESCOMP:b4b-devfrom
samsrabin:refactor-cropphase
Jan 27, 2026
Merged

Factor out new subroutine CropPhase_OnePatch#3713
samsrabin merged 4 commits into
ESCOMP:b4b-devfrom
samsrabin:refactor-cropphase

Conversation

@samsrabin
Copy link
Copy Markdown
Member

Description of changes

Will be necessary for #3633.

Specific notes

Contributors other than yourself, if any: None

CTSM Issues Fixed:

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

Testing performed, if any: clm_short tests are b4b.

@samsrabin samsrabin added this to the ctsm6.0.0 (code freeze) milestone Jan 22, 2026
@samsrabin samsrabin self-assigned this Jan 22, 2026
@samsrabin samsrabin added enhancement new capability or improved behavior of existing capability code health improving internal code structure to make easier to maintain (sustainability) b4b bit-for-bit blocker another issue/PR depends on this one labels Jan 22, 2026
@samsrabin
Copy link
Copy Markdown
Member Author

Blocks #3633.

@samsrabin samsrabin added PR status: awaiting review Work on this PR is paused while waiting for review. PR status: needs testing test: aux_clm Pass aux_clm suite before merging labels Jan 22, 2026
@samsrabin samsrabin added the next this should get some attention in the next week or two. Normally each Thursday SE meeting. label Jan 22, 2026
Copy link
Copy Markdown
Contributor

@wwieder wwieder left a comment

Choose a reason for hiding this comment

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

Assuming this is b4b it seems OK to me. Wonder if other's with a more refined SE want to weigh in.

Comment thread src/biogeochem/CNPhenologyMod.F90 Outdated
samsrabin and others added 2 commits January 23, 2026 15:40
@github-project-automation github-project-automation Bot moved this from In progress - b4b-dev to In progress - master in CTSM: Upcoming tags Jan 23, 2026
@samsrabin samsrabin moved this to In progress - b4b-dev in CTSM: Upcoming tags Jan 23, 2026
@samsrabin samsrabin moved this from In progress - master to In progress - b4b-dev in CTSM: Upcoming tags Jan 23, 2026
@samsrabin samsrabin removed PR status: awaiting review Work on this PR is paused while waiting for review. PR status: needs testing labels Jan 23, 2026
@samsrabin samsrabin merged commit f90f9f1 into ESCOMP:b4b-dev Jan 27, 2026
4 checks passed
@github-project-automation github-project-automation Bot moved this from In progress - b4b-dev to Done (non release/external) in CTSM: Upcoming tags Jan 27, 2026
@samsrabin samsrabin removed the next this should get some attention in the next week or two. Normally each Thursday SE meeting. label Feb 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

b4b bit-for-bit blocker another issue/PR depends on this one code health improving internal code structure to make easier to maintain (sustainability) enhancement new capability or improved behavior of existing capability size: small test: aux_clm Pass aux_clm suite before merging

Projects

Status: Done (non release/external)

Development

Successfully merging this pull request may close these issues.

3 participants