-
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.
Merge branch 'track_cwd_in_fluxes' from ckoven Added yet more history dims: CWD, fuel, canopy, canopy/leaf, and canopy/leaf/pft Added more dimensions to facilitate introspection of the model via the history files. Two major areas here: * 2 dimensions for the dead stuff: a CWD size-class dimension, and a fuel dimension (which is the CWD sizes plus 2 spots for leaves and grass). * 3 new dimensions across the canopy vertical gradients to facilitate figuring out the light environment. most of the raw variables here are indexed by canopy level, leaf level, and pft, so I made a gigantic dimensions which multiplexes all three of these into a single dimension. since this is big (2x40x2 =160 currently with ed_npfts = 2, but will grow rapidly once we add more PFTs), I made two smaller dimensions as well: one that is canopy layer by leaf layer (so length 2x40 = 80) that averages the light environment across PFTs (and ought to be sufficient/overkill for most needs in the long run) and one that is just canopy dimension (length 2) for looking at things like the light environment of the top level of leaves for each canopy level, and which could also be used to collapse some of the canopy-level diagnostics that I had previously defined just as 1d onto so that they use 1 variable instead of 2. Also added more diagnostic fluxes to track litter input fluxes due to disturbance and termination, which were previously being passed straight from live to dead pools without tracking the fluxes, and which are now still passed directly from live to dead pools, but, separately are being tracked and added to the litter_in and more resolved fluxes. separately, i fixed unit errors on some of the litter_in, litter_out, and other fluxes, but I've reverted the ones to litter_in and litter_out to pass regression tests (since those two variables are in the test) and will submit a second PR with just the fixes to that shortly. Added a bunch of variables on these new dimensions: * for the cwd we're tracking the stocks, input and output fluxes on the cwdsc dimensio, and for the fuel we are tracking fuel moisture on the fuel dimension * for the vertical canopy stuff we are tracking fabd, fabi, par and lai for sun and shade leaves across various slices of the new dims. for now these are all default on, but we can turn the larger ones off when we have a better sense of having fixed the canopy light environment. Renamed nlevcan to nlevleaf, because it is a better descriptor for what that variable represents -- the leaf sublayers within each canopy level -- but also mainly because CLM already has a variable nlevcan which collided with FATES's during the specification of the array lengths in histfilemod. updated 3/17 to note that @rgknox added one final dimension, which is the 'classic' ED dimension of cohort size bin multiplexed by patch age bin, and also did a bunch of cleanup, gave better names to the multiplexed dimension names, and replaced a bunch of inline unit changes with actual named unit conversion constants Added a named constant to clm history module to set length of dimension names. User interface changes?: no Code review: Some review already by @rgknox and @rosiealice, could do with some more. note that this is on top of PR #184 so best to wait until that is pulled so that these new changes are more apparent. Testing: ckoven: Test suite: Ed suite on lawrencium Test baseline: a5dc8da Test namelist changes: Test answer changes: bit for bit Test summary: all pass. andre: Test suite: ed - yellowstone gnu, intel, pgi hobart nag Test baseline: e99fcf9 Test namelist changes: none Test answer changes: bit for bit Test summary: all tests pass Test suite: clm_short - yellowstone gnu, intel, pgi Test baseline: clm4_5_12_r195 Test namelist changes: none Test answer changes: bit for bit Test summary: all tests pass
- Loading branch information
Showing
21 changed files
with
1,422 additions
and
385 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
Oops, something went wrong.