Skip to content
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

Adding control of leaf water potential on transpiration through inner leaf humidity (Hydro only) #736

Merged
merged 14 commits into from
Nov 2, 2021

Conversation

JunyanDing
Copy link
Contributor

Description:

Issue #726

Collaborators:

Ryan Knox, Chonggang Xu, Charlie Koven

Expectation of Answer Changes:

  1. incorporated the lwp on transpiration (see attached file for detail)
    StomaModification.pdf
  2. provide an option to use agross in BB stoma conductance model controlled by in code parameter use_agross

Checklist:

  • My change requires a change to the documentation.
  • I have updated the in-code documentation .AND. (the technical note .OR. the wiki) accordingly.
  • [ x] 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:


!!! Calculate anet, only exit iteration with negative anet when using anet in calculating gs
! this is version B
anet = agross - lmr
Copy link
Contributor

@rosiealice rosiealice Apr 9, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess I am a bit confused about why this section on agross is in this PR? Is it allied to the modification of gs by LWP, or orthogonal to it? If it's a seperate issue, should it be in a separate PR?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sort of. If using lwp to constrain the transpiration, it is more appropriate to use agross instead of anet

biogeophys/FatesPlantRespPhotosynthMod.F90 Outdated Show resolved Hide resolved
biogeophys/FatesPlantRespPhotosynthMod.F90 Outdated Show resolved Hide resolved
biogeophys/FatesPlantRespPhotosynthMod.F90 Outdated Show resolved Hide resolved
! Parameters
! ------------------------------------------------------------------------
! selection to use agross or anet in stomatal models, 1 - use agross, other values - use anet
integer,parameter :: use_agross = 1
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can this part of the parameter file?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

will add this to next round of parameters for the wish-list

real(r8) :: qs ! specific humidity at leaf surface
real(r8) :: qsat ! saturated specific humidity
real(r8) :: qsat_adj ! specific humidity adjusted by leaf water potential (g/kg)
real(r8) :: LWP_star ! leaf water potential scaling coefficient for inner leaf humidity
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you mention what value of 0 or 1 means for a better understanding of the code?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

comments were added

biogeophys/FatesPlantRespPhotosynthMod.F90 Outdated Show resolved Hide resolved
qsat = 0.622 * veg_esat / (can_press - 0.378 * veg_esat)
qsat_adj = qsat*LWP_star

if (k_lwp == 0 ) then
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since k_lwp is not an integer, it would be better to check if k_lwp is a very small number

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was treating k_lwp as an positive integer in my mind when coding. Will adjust this.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this has been updated and addressed

biogeophys/FatesPlantRespPhotosynthMod.F90 Outdated Show resolved Hide resolved
biogeophys/FatesPlantRespPhotosynthMod.F90 Outdated Show resolved Hide resolved
! then set stomata resistance to be very large to stop the transpiration or back flow of vapor
rstoma_out = rsmax0*100
else
rstoma_out = (qsat-qs)*( 1/gstoma + rb)/(qsat_adj - qs)-rb
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I personally do not like the add-on adjustment of leaf humidity on gs. I would prefer to updated the calculation of vpd (vapor pressure deficit from internal leave to air) or relative humidity (humidity in the air/humidity inside leave) for the two type of models. However, I think it is also justifiable as the above two models are fitted to air VPD assuming full saturation inside leaves. Not clear preference.....

remove the agross-anet option from this pull request, and put in a separate PR.
@glemieux glemieux assigned rgknox and unassigned rgknox Sep 20, 2021
…lew away old leaf-humidity stuff in FatesPlantRespPhotosynthMod because of so many differences and conflicts. Will re-apply manually looking at original diff with base
@rgknox
Copy link
Contributor

rgknox commented Nov 2, 2021

Tests look good, only known fails, b4b with base.

rgknox/tests_1102-100244ch> ./cs.status | grep FAIL
    FAIL SMS_Lm13.1x1_brazil.I2000Clm50FatesCruRsGs.cheyenne_gnu.clm-FatesColdDef TPUTCOMP Error: TPUTCOMP: Computation time increase > 25% from baseline
  ERS_Ld30.f45_f45_mg37.I2000Clm50FatesCruRsGs.cheyenne_intel.clm-FatesColdDefReducedComplexSatPhen (Overall: FAIL) details:
    FAIL ERS_Ld30.f45_f45_mg37.I2000Clm50FatesCruRsGs.cheyenne_intel.clm-FatesColdDefReducedComplexSatPhen RUN time=60
  ERS_Lm12.1x1_brazil.I2000Clm50FatesCruRsGs.cheyenne_intel.clm-FatesFireLightningPopDens (Overall: FAIL) details:
    FAIL ERS_Lm12.1x1_brazil.I2000Clm50FatesCruRsGs.cheyenne_intel.clm-FatesFireLightningPopDens COMPARE_base_rest
    FAIL ERS_Lm12.1x1_brazil.I2000Clm50FatesCruRsGs.cheyenne_intel.clm-FatesFireLightningPopDens TPUTCOMP Error: TPUTCOMP: Computation time increase > 25% from baseline

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.

6 participants