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

Leaf orientation index range commented out #1001

Closed
adrifoster opened this issue Mar 16, 2023 · 3 comments
Closed

Leaf orientation index range commented out #1001

adrifoster opened this issue Mar 16, 2023 · 3 comments
Assignees

Comments

@adrifoster
Copy link
Contributor

In EDSurfaceAlbedoMod a line capping the PFT parameter fates_leaf_xl (leaf orientation index) has been commented out. According to Gordon Bonan's textbook (page 238), "this equation is restricted to -0.4 < χl < 0.6"

Based off of many of my runs failing (for solar radiation balance check errors) when this parameter is outside those ranges (as well as within... but perhaps a separate issue), I think we should put this capping back.

@jkshuman
Copy link
Contributor

Thanks for noting this @adrifoster and providing the reference to Gordon's book. Seems like your testing supports the change.

@adrifoster
Copy link
Contributor Author

adrifoster commented Mar 16, 2023

Suggestion from @JessicaNeedham: we should also perhaps put something to this effect in the parameter's long name in the parameter file. Just so folks know we are capping this parameter.

@rgknox
Copy link
Contributor

rgknox commented Mar 16, 2023

@adrifoster I like @JessicaNeedham 's idea, and I'd also like to pair that with a graceful fail mechanism during the parameter checks in initialization if a user sets a value of xl outside this range. If we do this, maybe we don't apply capping in the radiation call, so the two methods don't diverge

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

No branches or pull requests

3 participants