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

Outlines/outfalls overview #227

Open
3 of 14 tasks
barneydobson opened this issue Jun 24, 2024 · 0 comments
Open
3 of 14 tasks

Outlines/outfalls overview #227

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

Comments

@barneydobson
Copy link
Collaborator

barneydobson commented Jun 24, 2024

There are a lot of issues that relate outline/outfall derivation. Furthermore, this choice has an enormous impact on the derived network. There is a sense that the current method needs improvement. In this PR I will explain the current processes and bring together existing issues. Here we can also discuss proposed improvements.

Three current options exist (two on swmmanywhere - which primarily take place in graph_utilities.py/clip_to_catchments):

  • Clip to subbasin (i.e., parameters.py/SubcatchmentDerivation.subbasin_clip_method='subbasin'). Where subbasin is derived either using pyflwdir.snap or, if a non-None parameter is provided, using SubcatchmentDerivation.subbasin_streamorder=<int>. In this method, all potential pipe carrying links that span two subbasins are removed - thus flows always stay within a subbasin.
  • Community threshold (SubcatchmentDerivation.subbasin_clip_method='community'). Subbasins are derived, as are communities using nx.louvain_communities. Communities with less than subcatchment_derivation.subbasin_membership proportion of nodes in a subbasin have their links to all other nodes in that subbasin removed.
  • Bypass this issue by providing a user defined outline, and removing all nodes/catchments outside of that outline (see trim_to_real here ).

In addition to the processes above, if there are rivers near the street network, then further processing to identify which outfalls should be used takes place in graph_utilities.py/identify_outlets.

Grouping these issues below:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancements feature Adding a new functionality, small or large
Projects
None yet
Development

No branches or pull requests

1 participant