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

Match streets to rivers with attention to underlying DEM #201

Open
1 of 4 tasks
Tracked by #227
barneydobson opened this issue Jun 3, 2024 · 0 comments
Open
1 of 4 tasks
Tracked by #227

Match streets to rivers with attention to underlying DEM #201

barneydobson opened this issue Jun 3, 2024 · 0 comments
Labels
feature Adding a new functionality, small or large

Comments

@barneydobson
Copy link
Collaborator

barneydobson commented Jun 3, 2024

I don't think I agree with the logic here. Using only a buffer distance to identify a water body that a node can drain to is not necessarily hydraulically feasible. A raindrop trace operation needs to be done to ensure that a water body exists at the downstream of nodes (so it can drain to) and, if desired, within an allowable distance. You can do this with pyflwdir.

I think I should've phrased my question better. I understand what this specific set of functions is doing, but my question was: How does it fit in the overall workflow? When do these get applied, and what information is included at this stage? IMHO, only using elevation to determine the outlet location is impractical. In practice, a lot more considerations, especially environmental ones, go into selecting them. So, I am trying to understand the implications of relying solely on the elevation data for this purpose.

Off the top of my head, two of the most important factors are the min discharge velocity at the outfall (in the US, depending on the municipality it can be 5, 10, or 20 ft/s) and existence of fish habitat and such in the water body.

Originally posted by @cheginit in #144 (comment)

I think in general can use this issue to discuss which factors are important for outfall identification:

  • Network topology (included with Improve identify outlets #144 )
  • DEM (as mentioned above)
  • Environmental considerations (@cheginit do you mean by this like flow in the receiving water body?)
  • Minimum velocity required

.. any other factors?

@barneydobson barneydobson added the feature Adding a new functionality, small or large label Jun 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Adding a new functionality, small or large
Projects
None yet
Development

No branches or pull requests

1 participant