add interflow - #62
Merged
peterlafollette merged 12 commits intoJun 13, 2026
Merged
Conversation
…of lateral fluxes
ajkhattak
reviewed
Jun 9, 2026
ajkhattak
left a comment
Collaborator
There was a problem hiding this comment.
Peter, thanks for adding the lateral flow process to CASAM. From a technical standpoint, it looks solid overall. There are a few sections that I wasn't able to fully follow, so let's tag up to briefly discuss them.
One suggestion I have is to use more descriptive variable names and add documentation/comments for the new functions you've implemented. This would make the code easier to understand and maintain.
Thanks!
added 7 commits
June 10, 2026 16:54
when interflow is on, it sometimes would be intense enough to make theta want to go below theta_r and foist the mass balance error into AET. Now lateral fluxes will be reduced such that they will not imply to an impossible target soil profile.
…er interflow extraction will not greatly exceed the threshold for interflow generation
updated conceptual reservoir parameters to be more descriptive, for example the parameter a is now a_con_res. Further updated parameters and fluxes having to do with lateral subsurface flow to have interflow in the name. Legacy parameter names are still accepted. Also added more commenting in the code about log mode.
Contributor
Author
|
Replied to / resolved all comments and met about them -- most salient changes are updated clearer naming conventions, better guard to prevent interflow from extracting too much water for any given time step, more commenting in code, and updated readme documentation. |
ajkhattak
approved these changes
Jun 13, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR adds the capacity for interflow, such that water stored in the vadose zone (LGAR domain) can contribute directly to streamflow. A WF contributes to streamflow if its capillary head is above a new specified parameter, lateral_flow_psi_threshold, and another new parameter lateral_flow_factor modifies lateral flow contributions to streamflow. Interflow is proportional to the hydraulic conductivity of a wetting front. Interflow is enabled only when both parameters are specified. Flux caching is disabled during timesteps where lateral flow is active and eligible fronts are present. The lateral_flow_factor parameter should be interpreted as a lumped effective lateral conductance term that can include anisotropy, hydraulic gradient, drainage length, and catchment connectivity effects.
Additions
Removals
Changes
Testing
Screenshots
Notes
Todos
Checklist
Testing checklist
Target Environment support
Accessibility
Other