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

Update set_soils() to work with rSOILWAT2 before and after v6.0.0 #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dschlaep
Copy link
Member

@dschlaep dschlaep commented Aug 9, 2023

  • rSOILWAT2 v6.0.0 introduced custom soil water retention curves (via a new slot that contains parameters for each soil layers) - they must be kept up-to-date when soil information changes

  • rSOILWAT2 provides the method swSoils_Layers() to retrieve and replace soil information -- and to up-date soil water retention curve parameters

  • this also introduces a hack to fix evco and trco parameters which were not meeting requirements because the set_soils() function repeats the 3rd and 8th soil layer information; ideally, these parameters should be derived anew (replacing this hack)

- rSOILWAT2 v6.0.0 introduced custom soil water retention curves (via a new slot that contains parameters for each soil layers) - they must be kept up-to-date when soil information changes
- rSOILWAT2 provides the method `swSoils_Layers()` to retrieve and replace soil information -- and to up-date soil water retention curve parameters

- this commit also introduces a hack to fix evco and trco parameters which were not meeting requirements because the `set_soils()` function repeats the 3rd and 8th soil layer information; ideally, these parameters should be derived anew (replacing this hack)
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.

1 participant