-
Notifications
You must be signed in to change notification settings - Fork 92
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Initial carbon linkages between ED and soil BGC
Merge andre-ed-v0.1.0-buggy-science into master initial implementation of the carbon linkages from ED to the CLM soil biogeochemistry cycle. I originally did this in the CLM SVN, which Ben pulled into the NGT github on the "buggy science" branch. The purpose of this code is to take all the litter that flows out of ED, aggregated onto column-level variables and disaggregated vertically and into litter decomposability pools, and then send this to the BGC code that is shared with the rest of the model. The code behavior should be identical to the master for the variables that are output in master, but there is now a much larger set of carbon cycle variables that were not output before. New output variables include soil and litter carbon pools, combined ED + BGC pools (TOTECOSYSC), and a bunch of fluxes (heterotrophic respiration, NEP, NBP, etc). I've also included some extra diagnostics, including high-frequency NPP on the column level (current standard NPP is daily-averaged on patch level, which I've left as-is). For carbon balance, I am now tracking all fluxes into the ED pools and the BGC pools, separately and together, over each day (since ED fluxes are resolved on the daily timestep and CLM fluxes are resolved on the half-hourly timestep). Carbon balance errors are not trivially small, so instead of implementing a tolerance-checking crasher I am just writing the errors to history for now. When we get these smaller we can replace them with logic that crashes when a tolerance is exceeded. I find some of the behavior hard to check in the history files because of issues #24 and #26, but otherwise things seem reasonable. I have avoided crashing bugs associated with issue #30 but not actually tried to solve that. Code Review: C Koven, R Knox Closes: #35 Test suite: ed - lawrencium pass; ed - yellowstone intel, pgi, gnu Test baseline: d8a9ee5 Test namelist changes: yes Test answer changes: yes Test summary: expected fails in f19 and f09 restart, answer changing and nl changes as expected.
- Loading branch information
Showing
6 changed files
with
1,155 additions
and
136 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.