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

Potential Issue with Velocity Calculation for Compound Channel Flow in module_channel_routing.F #772

Open
mrosicky opened this issue Sep 23, 2024 · 0 comments
Assignees
Labels
physics issues related to physics components question Further information is requested

Comments

@mrosicky
Copy link

While reviewing the channel routing implementation in module_channel_routing.F, I encountered what may be an issue regarding the computation of flow velocity. I believe the current approach might not properly account for the case when flow spills into the floodplain.

The depth variable h, derived from the MC equation, appears to represent the total flow depth, which includes both the main channel and floodplain when there is overflow (i.e., h > bfd where bfd is the bankfull depth) at line 337 and line 354.

However, average velocity is calculated using Manning's equation at line 498, where R is the hydraulic radius based on h at line 497. This equation applies to flow in the main channel only, not the compound flow that includes the floodplain.

If h includes the floodplain depth, the current velocity calculation might be incorrect, as it does not separately account for the floodplain's resistance or hydraulic characteristics.

Expected Behavior

The velocity is calculated with separate components for the main channel and floodplain to ensure accurate modeling of compound channel flow, then combined.

Possible Solution

Calculate separate velocities for the main channel and the floodplain, using their respective hydraulic parameters (e.g., AREAC, WPC). Then, combine the velocities for an accurate representation of the total flow across both the main channel and floodplain.
@aubreyd aubreyd added question Further information is requested physics issues related to physics components labels Oct 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
physics issues related to physics components question Further information is requested
Projects
None yet
Development

No branches or pull requests

3 participants