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

Guardrails #1099

Closed
rgknox opened this issue Oct 16, 2023 · 2 comments
Closed

Guardrails #1099

rgknox opened this issue Oct 16, 2023 · 2 comments

Comments

@rgknox
Copy link
Contributor

rgknox commented Oct 16, 2023

Issue #1090 revealed the possibility that some allometries could create unusual plant sizes or organ proportions. In that example, some trees became so large, that the stature growth integrator had a tough time generating precise results.

During review of this issue, several developers thought it could be useful, if we come up with strategies to apply optional guardrails to different aspects of the model. Ideally, the guardrails are there to prevent very unusual, and hopefully very seldom model states from occurring and breaking the model.

For instance, in the allometry example, we could set caps on maximum sizes. When the maximum size is reached, we would shunt all NPP to either reproduction, exudation, or respire it away, after the plant has replaced turnover. This is just an example. This type of behavior would kick in only for a plant or organ size that is far beyond the maximum expected. Someone may want this type of behavior if they are running a ensembles of parameters, and it is more desirable that all the runs complete, even if they have strange results, rather than one of their runs failing and breaking their workflow.

Feel free to use this space to come up with ideas for different guardrails, and in what contexts it would be useful to apply them. Note that we could keep guardrails default off, as a user defined parameter.

cc @ckoven

@rgknox
Copy link
Contributor Author

rgknox commented Oct 25, 2023

After some discussion, there is some thought that current issues with integration errors relate to growth, may have something to do with capping in allometry and/or the use of allometries that rely on a sudden change in allocations. An example is hitting a maximum size threshold and pivoting to growing the stem diameter instead of stem and height simultaneously. So instead of applying these guardrails, we will wait and see if phasing out capping-style and sudden change style allometry in favor of asymptotic and smooth functions gets rid of these problems.

@rgknox
Copy link
Contributor Author

rgknox commented Oct 26, 2023

closing feel, free to reopen

@rgknox rgknox closed this as completed Oct 26, 2023
@github-project-automation github-project-automation bot moved this from ❕Todo to ✔ Done in FATES issue board Oct 26, 2023
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

1 participant