Skip to content

Nbp nobalancecheck#20

Merged
rosiealice merged 21 commits intoNorESMhub:noresmfrom
rosiealice:nbp_nobalancechec
Jun 16, 2025
Merged

Nbp nobalancecheck#20
rosiealice merged 21 commits intoNorESMhub:noresmfrom
rosiealice:nbp_nobalancechec

Conversation

@rosiealice
Copy link
Collaborator

These changes introduce the capability for FATES to write NBP (net biome productivity) fluxes to the atmosphere, inclusive of fire, harvest, grazing and product pool decay carbon flux.

This PR is linked to the CTSM-side PR (NorESMhub/CTSM#140)

Noting that this code neither creates nor passes the gridcell level balance check on account of timestepping complexities that need to be resolved in a later PR., likely NGEET#137

To recall:
NBP =.NEP - C_fire - C_grazing - C_productdecay

where fire and grazing fluxes are now output from FATES, and product decay (a gridcell property) is calculated in CTSM.

NEP is the Net Ecosystem Productivity, defined as
NEP = GPP - Reco

where GPP is the gross photosynthetic productivity and Reco is total ecosystem respiration inclusive of plant (Rauto) and soil (Rh)

Solves.
NGEET#163

Description:

Collaborators:

@ckoven
@maritsandstad
@mvdebolskiy
@rgknox

Expectation of Answer Changes:

This should not chance answers EXCEPT for the addition of the new NBP output variable, and the FCO2 flux variable which was peviously zero.

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:

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

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

FATES baseline hash-tag:

Test Output:

@mvdebolskiy mvdebolskiy mentioned this pull request Jun 8, 2025
16 tasks
@rosiealice
Copy link
Collaborator Author

All the aux_clm_noresm tests pass, so merging this.

@rosiealice rosiealice merged commit 3ff29e5 into NorESMhub:noresm Jun 16, 2025
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.

3 participants