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

fix typo in modify_fates_paramfile.py #1275

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

fix typo in modify_fates_paramfile.py #1275

wants to merge 1 commit into from

Conversation

ckoven
Copy link
Contributor

@ckoven ckoven commented Oct 30, 2024

The dimension name for land use in the parameter file didn't match what it was listed as in modify_fates_paramfile.py, so this corrects that.

Description:

n/a

Collaborators:

n/a

Expectation of Answer Changes:

None, python code only.

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:

no testing because only python code.

@glemieux glemieux added the type: tools This PR adds or updates support tools. No regression testing necessary. label Nov 18, 2024
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
Development

Successfully merging this pull request may close these issues.

2 participants