-
Notifications
You must be signed in to change notification settings - Fork 92
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
Improve the plant hydro codes to accomodate dynamic forest #440
Merged
rgknox
merged 63 commits into
NGEET:master
from
xuchongang:xuchongang/fates_hydro_improve_ready_to_merge
Jan 14, 2019
Merged
Changes from 58 commits
Commits
Show all changes
63 commits
Select commit
Hold shift + click to select a range
4183e31
initial set of add'l data structures and code for plant growth & turn…
e8cefc0
rounded out FuseCohortHydraulics and CopyCohortHydraulics with the mi…
3e9f814
added some hydraulics variables to dump_cohort(). Beware, there is no…
ded6ccc
shuffled order of a few cohort-level hydraulics variables, for visual…
c8f8243
added numpft to InitHydraulicsDerived to enable initializing derived …
931ca5f
uncommented call to updateSizeDepRhizHydStates
97a0dc6
added soil water content by layer and rhizosphere shell, h2osoi_shsl.…
3cd6830
added calculation of derived pinot pft-specific hydraulics parameter,…
c5f2edf
added call to new subroutine RecruitWaterStorage(), which calculates …
6be3995
added subroutine RecruitWaterStorage which caluclates site-level amou…
257c1e4
added H2OVEG_RECRUIT as a history output variable
9134ac4
added site-level variable h2oveg_recruit
a8481ea
adds a call to AccumulateMortalityWaterStorage from within terminate_…
eb5d4cb
update the recruit water uptake,veg water storage and a typo correction
xuchongang 8a0a1cf
Merge branch 'xuchongang/fix_hydraulic_bug_ready_to_merge' of github.…
xuchongang 9abc96b
Merge branch 'master' of github.com:NGEET/fates into xuchongang/fates…
xuchongang 0a96285
update the HYDRO warning
xuchongang 2933859
Upudate the dump_cohort statement for hydro
xuchongang 45c31fe
using associated(ccohort_hydr%th_aroot) instead of hlm_use_planthydro
xuchongang 790db87
update the fill rhizosphere drainage filling for mass balance
xuchongang 8162bc1
update the rhizosphere drainage refilling for improvement of mass bal…
xuchongang b9a88e4
avoid the reference to site%hydr for Fates History when hydro is not …
xuchongang a7c674e
Merge branch 'xuchongang/fates_hydro_improve_fillrhiz' of github.com:…
xuchongang dc04ffe
update the dump_cohort for hydro
xuchongang 13ada94
simply the rhizosphere kmax code by avoiding duplicate calculations
xuchongang 26e3db8
turn on the default sapwood area calculation for hydro
xuchongang aa86820
Starting to scratch out variable dependencies for hydro restarts
rgknox 977c6c7
Spacing out and listing hydro variables that will be added to restarts.
rgknox b84261a
First pass at incorporating hydraulics variables into restarts
rgknox 37291fc
syntax and bug fixes for hydro restarts, code compiles, un-tested
rgknox ffd5e4f
SPlitting up hydraulic props routine
rgknox 56de482
update the c_area and treelai for fusecohort in hydraulics
xuchongang 0a2e40d
use the profile-based leaf area for hydraulics for code consistency
xuchongang 57beba2
Merge branch 'xuchongang/fates_hydro_improve_fillrhiz' of github.com:…
xuchongang e6679a8
Re-arranging calls to initialize hydro cohorts
rgknox 44d639d
Debugging new hydro restarts
rgknox 6abcbe6
update the leaf lai for hydro after fuse cohorts
xuchongang 9f262a6
track the error for hydro dynamics
xuchongang b529e5c
avoid the calls to update plant water when no recruitment happens in …
xuchongang 258acb6
add a debug flag to turn on and off warning for mass error checking
xuchongang cfbdb86
cleaned the codes on updating veg water storage and error terms
xuchongang 015533c
fates hydro restart refactors - syntax updates
rgknox a90d5e0
conflict resolution related to sapwood area calculation
rgknox 6481b73
Added some zero-ing initializations to hydraulics during restarts
rgknox 50e8141
Corrected hydro-restart initialization procedure
rgknox 757d93b
Removed length, node and volumes from shell restarts as they are auto…
rgknox 9d70ddd
Added debug protection to an abundant water balance error write state…
rgknox 2254e76
Merge pull request #3 from rgknox/rgknox-cxu-dynamic-hydro-restart-me…
xuchongang a250245
Add the recruitment water constrain based on water availability in t…
xuchongang 8b6371f
update the recruit water constrain for hydro
xuchongang 8e03126
update the recruit water constrain within plant hydro
xuchongang 22b84c5
update the hydro codes to avoid positive water potentials
xuchongang 4a20a17
update the recruitment constraint for parteh
xuchongang be3ae7a
add a new default parameter file for hydro
xuchongang d2c8d8b
add the hydraulic failure mortality based on fractional loss of condu…
xuchongang 8fcbadd
update the parameter files for hydraulic failure mortality
xuchongang 78b28ab
correct a typo of hf_flc_threshold
xuchongang 30f776d
moved a comment on hf mortality for easier readibility of codes
xuchongang f0bce93
Reset default settings for phenology in EDTypes
rgknox 657473e
Fixes for history and restarts, making sure that hydro diagostics are…
rgknox 8971025
Merge pull request #6 from rgknox/xuchongang/fates_hydro_improve_read…
xuchongang 4f0310d
Trival conflict resolutions with master
rgknox 911b539
Merge pull request #7 from rgknox/xuchongang/fates_hydro_improve_read…
xuchongang File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
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
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.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
At first I thought maybe we should consider having just one parameter here, generically defined as "hydraulic failure threshold". But it does appear that the units of hf_sm_threshold and hf_flc_threshold are completely different. One is soil moisture and the other is conductivity. I think I like them separated into two different values. Maybe we should add a note in the parameter file meta-data and the parameter definitions "in-code" as to where they are relevant though.