Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

vegetation bin indexing #1364

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

vegetation bin indexing #1364

wants to merge 2 commits into from

Conversation

rgknox
Copy link
Contributor

@rgknox rgknox commented Mar 18, 2025

Description:

This set of changes introduces some consistency improvements to how we determine the number of vegetation layers, and how we access and use those indices. Of note, the lower bound of this array has been changed to 0.

Collaborators:

@ckoven

Expectation of Answer Changes:

Not sure. This may be a bug fix as well. Looking again.

Checklist

If this is your first time contributing, please read the CONTRIBUTING document.

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

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:

@glemieux glemieux added the type: refactor Restructures code without changing functionality label Mar 24, 2025
@glemieux glemieux moved this to Finding Reviewers in FATES Pull Request Planning and Status Mar 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: refactor Restructures code without changing functionality
Projects
Status: Finding Reviewers
Development

Successfully merging this pull request may close these issues.

2 participants