Skip to content

Taking out option that sets vcmax25top etc to zero in the case of no …#1537

Merged
rgknox merged 2 commits intoNGEET:mainfrom
rgknox:mirror-zeroleaf-restartbug
Mar 24, 2026
Merged

Taking out option that sets vcmax25top etc to zero in the case of no …#1537
rgknox merged 2 commits intoNGEET:mainfrom
rgknox:mirror-zeroleaf-restartbug

Conversation

@rgknox
Copy link
Copy Markdown
Contributor

@rgknox rgknox commented Feb 26, 2026

This is a cherry-pick of: NorESMhub#54

Issued by @maritsandstad and @mvertens, with help from @mvdebolskiy, @rgknox and @rosiealice

See issue:
NorESMhub/CTSM#200

All checklist items must be checked to enable merging this pull request:

Contributor

  • The in-code documentation has been updated with descriptive comments
  • The documentation has been assessed to determine if updates are necessary

Integrator

  • FATES PASS/FAIL regression tests were run
  • Evaluation of test results for answer changes was performed and results provided
  • FATES-CLM6 Code Freeze: satellite phenology regression tests are b4b

If satellite phenology regressions are not b4b, please hold merge and notify the FATES development team.

Documentation

Test Results:

CTSM (or) E3SM (specify which) test hash-tag:

CTSM (or) E3SM (specify which) baseline hash-tag:

FATES baseline hash-tag:

Test Output:

@rgknox
Copy link
Copy Markdown
Contributor Author

rgknox commented Mar 24, 2026

running tests: /.../tests_0323-184529de
Tests pass, SP B4B, expected diffs on other baselines

@rgknox rgknox merged commit 326607f into NGEET:main Mar 24, 2026
1 check passed
@github-project-automation github-project-automation Bot moved this from Finding Reviewers to Ready to Integrate in FATES Pull Request Planning and Status Mar 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

3 participants