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

Local inertial 1D-2D: inflow, reservoirs and lakes #213

Merged
merged 12 commits into from
Jul 13, 2022

Conversation

verseve
Copy link
Member

@verseve verseve commented Jun 27, 2022

Inflow (external) added to local inertial 1D-2D model.
Fixed couple of issues related to reservoir and lake locations, and added check (limit flow (set to zero)) when water is not available.

add river inflow to 1D-2D local inertial model
in case of reservoir or lake location, q_av remained zero
reservoirs or lakes now also possible as part of local inertial 1D-2D model.
add path of CSV file with rating curve or storage curve to logging
water depth h for reservoir and lake locations set at zero as fixed boundary instead of bankfull depth, upstream flow direction is not allowed then.
fix control flow reservoir and lake locations as part of shallow water (boundary points: update volume not required).
@verseve verseve marked this pull request as draft June 29, 2022 12:12
@verseve verseve marked this pull request as ready for review June 29, 2022 14:36
Copy link
Contributor

@JoostBuitink JoostBuitink left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice fix, and nice to separate the lakes and reservoir section in the docs! Has quite some effect in basins with reservoirs and/or lakes, such as the Rhine

@verseve verseve merged commit c3af84d into master Jul 13, 2022
@verseve verseve deleted the inflow-local-inertial branch July 13, 2022 09:11
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.

2 participants