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

Update FatesPFTIndexSwapper.py to work with landuse dimension #1145

Merged
merged 1 commit into from
Jan 8, 2024

Conversation

JessicaNeedham
Copy link
Contributor

@JessicaNeedham JessicaNeedham commented Jan 4, 2024

The new land use class dimension in the fates parameter file means that FatesPFTIndexSwapper.py needs to be updated.

Collaborators:

Expectation of Answer Changes:

These are minor changes only to FatesPFTIndexSwapper.py and will not affect FATES runs.

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:

@JessicaNeedham JessicaNeedham added the type: tools This PR adds or updates support tools. No regression testing necessary. label Jan 4, 2024
@glemieux
Copy link
Contributor

glemieux commented Jan 8, 2024

I conducted a quick check with @JessicaNeedham help and the tools appears to work as expected.

@glemieux glemieux merged commit 698a8df into NGEET:main Jan 8, 2024
1 check was pending
@JessicaNeedham JessicaNeedham deleted the jfn-fatespftindexswapper-fix branch January 11, 2024 21:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: tools This PR adds or updates support tools. No regression testing necessary.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants