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

Reservoir inflow/outflow mapping difference in KW and LI routing #436

Open
2 tasks done
atsiokanos opened this issue Jul 5, 2024 · 1 comment
Open
2 tasks done
Labels
documentation Improvements or additions to documentation v1.0 Activities related to v1.0

Comments

@atsiokanos
Copy link

Wflow version checks

  • I have checked that this issue has not already been reported.

  • I have checked that this bug exists on the latest version of Wflow.

Reproducible Example

NA

Current behaviour

It seems that the parameter "lateral.river.q_av" mapped at the reservoir locations ("wflow_reservoirlocs") represents inflows in kinematic wave models (identical to "lateral.river.reservoir.inflow") and outflows in local inertial (2d) model (identical to "lateral.river.reservoir.totaloutflow")

Desired behaviour

Keep them consistent or document the differences.

Additional Context

No response

@atsiokanos atsiokanos added the bug Something isn't working label Jul 5, 2024
@verseve verseve added documentation Improvements or additions to documentation and removed bug Something isn't working labels Jul 8, 2024
@verseve
Copy link
Contributor

verseve commented Jul 8, 2024

This is not a bug, but caused by a difference in routing concept (local inertial vs. kinematic wave). For the kinematic wave we decided to solve river flow up to and including the reservoir location, which is more accurate for relatively small reservoirs. For the local inertial routing, reservoir locations are handled as boundary conditions: with a fixed h for the reservoir location (node) and reservoir outflow is copied to the downstream edge of that node.

It is not recommended to use q_av for the computation of reservoir inflow, because reservoir inflow also consists of overland and lateral subsurface flow.

Good suggestion to document this!

@verseve verseve added the v1.0 Activities related to v1.0 label Jul 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation v1.0 Activities related to v1.0
Projects
None yet
Development

No branches or pull requests

2 participants