Skip to content

Add new FATES crop PFT and HLM PFTs#18

Merged
rosiealice merged 4 commits intorosiealice:jessie_ENT_allometry_updatesfrom
JessicaNeedham:jessie_ENT_allometry_updates
Feb 27, 2026
Merged

Add new FATES crop PFT and HLM PFTs#18
rosiealice merged 4 commits intorosiealice:jessie_ENT_allometry_updatesfrom
JessicaNeedham:jessie_ENT_allometry_updates

Conversation

@JessicaNeedham
Copy link

@JessicaNeedham JessicaNeedham commented Feb 23, 2026

Description:

Combine various changes to parameter file. This brings in changes from PR #53 and also adds the two extra HLM PFTs to go with the 16 hlm PFT surface file. Diff looks much bigger than it is, I think variable order is getting mixed up from FatesPFTIndexSwapper.

Collaborators:

@rosiealice

Expectation of Answer Changes:

Checklist:

  • My change requires a change to the documentation.
  • I have updated the in-code documentation .AND. (the technical note .OR. the wiki) accordingly.
  • I have read the CONTRIBUTING document.
  • FATES PASS/FAIL regression tests were run
  • If answers were expected to change, evaluation was performed and provided

Test Results:

Short 5 year test suggests this is working as expected.
Nocomp PFT area:
image

and PFT GPP:
image

Copy link
Owner

@rosiealice rosiealice left a comment

Choose a reason for hiding this comment

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

I guess Charlie's tool reorders the variables, hence the change in the order of the headers? I saw this when I tried to do this earlier as well. Otherwise this looks as intended.

@rosiealice rosiealice merged commit 4c2505e into rosiealice:jessie_ENT_allometry_updates Feb 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants