Skip to content

update hydraulic conductivity scaling model#1565

Open
xuchongang wants to merge 5 commits into
NGEET:mainfrom
xuchongang:xuchongang/taper_update
Open

update hydraulic conductivity scaling model#1565
xuchongang wants to merge 5 commits into
NGEET:mainfrom
xuchongang:xuchongang/taper_update

Conversation

@xuchongang
Copy link
Copy Markdown
Contributor

@xuchongang xuchongang commented May 7, 2026

Taper improvement.pdf

This pull request update the petiole length parameter to match the scaling of conduit size from Olson 2020 (https://nph.onlinelibrary.wiley.com/doi/10.1111/nph.16961)

Description:

The original scaling model has a much larger scaling of conduit size compared to the data from Olson 2020. To resolve this issue, I have added a new taper function based on Olson 2020. Please see attached the derivation of the taper components. Our hydraulic model optimization showed that this version of the taper function should be much closer to observations. This could also resolve the issue of size distribution of water potential. By lowering the hydraulic conductivity for taller trees, it will give a higher water stress for larger size of tree and thus higher mortality.

Collaborators:

@bchristo

Expectation of Answer Changes:

By change this values, we will see more reasonable scaling and make the model easier to fit to observed sapflow.

Description of generative AI usage (as necessary)

No AI used.

Checklist

If this is your first time contributing, please read the CONTRIBUTING document.

All checklist items must be checked to enable merging this pull request:

Contributor

  • [ x] The in-code documentation has been updated with descriptive comments
  • The documentation has been assessed to determine if updates are necessary
  • Describe use of generative AI (if necessary)

Integrator

  • FATES PASS/FAIL regression tests were run
  • Evaluation of test results for answer changes was performed and results provided
  • FATES-CLM6 Code Freeze: satellite phenology regression tests are b4b

If satellite phenology regressions are not b4b, please hold merge and notify the FATES development team.

Documentation

No need to change to the documentation.

Test Results:

See attached figures of scaling with the impact of change.

CTSM (or) E3SM (specify which) test hash-tag:

CTSM (or) E3SM (specify which) baseline hash-tag:

FATES baseline hash-tag:

Test Output:

… to match the scaling for conduit width from Olson 2020
@ckoven
Copy link
Copy Markdown
Contributor

ckoven commented May 7, 2026

Thanks Chonggang and really interesting. Should this be in the parameter file, rather than hard-coded though?

@xuchongang
Copy link
Copy Markdown
Contributor Author

xuchongang commented May 7, 2026 via email

@glemieux glemieux assigned rgknox and unassigned rgknox May 7, 2026
@glemieux glemieux requested a review from rgknox May 7, 2026 23:24
@xuchongang xuchongang changed the title update the petiole length parameter to match the scaling from Olson 2020 update hydraulic conductivity scaling model May 11, 2026
@xuchongang
Copy link
Copy Markdown
Contributor Author

@JunyanDing, @jennykowalcz , @yfang-pnnl , it would be great if you could review.

@rgknox, it would be great if you could do some test on the most recent version of FATES!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Finding Reviewers

Development

Successfully merging this pull request may close these issues.

4 participants