-
Notifications
You must be signed in to change notification settings - Fork 465
Add NBP, TOTECOSYSC, and related summary variables for FATES configurations #7231
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
Changes from all commits
856a4e8
e889591
7446da1
0c13d7c
587932c
9bc67dc
5195b10
968bef1
91d15a1
df3d75b
f1b74a6
7563787
5f1e4e4
d47039d
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
This file was deleted.
This file was deleted.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1 +1 @@ | ||
| fates_paramfile = '$CASEROOT/fates_params_twostream.nc' | ||
| fates_radiation_model = 'twostream' |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -274,6 +274,11 @@ subroutine EcosystemDynLeaching(bounds, num_soilc, filter_soilc, & | |
|
|
||
| call t_stop_lnd(event) | ||
|
|
||
| if (use_fates) then | ||
| call alm_fates%wrap_FatesAtmosphericCarbonFluxes(bounds, num_soilc, filter_soilc) | ||
| call alm_fates%wrap_FatesCarbonStocks(bounds, num_soilc, filter_soilc) | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @ckoven This call to wrap_FatesCarbonStocks over-writes the values for totecosysc (and others), which are filled above in col_cs_Summary(col_cs,bounds, num_soilc, filter_soilc). We should make sure we aren't filling the same variable in two different places (at least for similar indices). Option 1 is to use the col%is_fates filter to only update values in col_cs_Summary for non fates indices, and set to NaN for fates indices. Option 2, which I like more, is to have totecosysc only updated in col_cs_Summary, but have optional logic for FATES columns...
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I gave it a go of refactoring this, but ran into circular dependencies. I don't want the perfect to get in the way of the good, so I think this is fine. |
||
| endif | ||
|
|
||
| end subroutine EcosystemDynLeaching | ||
|
|
||
|
|
||
|
|
||
| +1 −1 | CODE_OF_CONDUCT.md | |
| +5 −5 | CONTRIBUTING.md | |
| +2 −2 | README.md | |
| +416 −977 | biogeochem/EDCanopyStructureMod.F90 | |
| +19 −13 | biogeochem/EDCohortDynamicsMod.F90 | |
| +7 −3 | biogeochem/EDPatchDynamicsMod.F90 | |
| +54 −4 | biogeochem/FatesPatchMod.F90 | |
| +19 −25 | main/EDMainMod.F90 | |
| +8 −8 | main/EDParamsMod.F90 | |
| +28 −25 | main/FatesHistoryInterfaceMod.F90 | |
| +1 −1 | main/FatesInterfaceMod.F90 | |
| +0 −2 | main/FatesInterfaceTypesMod.F90 | |
| +27 −3 | main/FatesRestartInterfaceMod.F90 | |
| +1 −1 | testing/unit_testing/sort_cohorts_test/test_SortCohorts.pf |
Uh oh!
There was an error while loading. Please reload this page.