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

Valley-bottom centerline #270

Closed
philipbaileynar opened this issue Apr 7, 2021 · 3 comments
Closed

Valley-bottom centerline #270

philipbaileynar opened this issue Apr 7, 2021 · 3 comments
Assignees
Labels
enhancement New feature or request pkg:VBET VBET python package

Comments

@philipbaileynar
Copy link
Contributor

We need to enhance VBET to produce a valley centerline. This will be used for chopping the valley bottom into sausages, and also for metrics in the reach typing work.

Ideas....

  1. Start by producing an intermediate dataset of all the input and output points.
    2. Don't mind if you do this via NetworkX or the NHD from/to node information.
    3. Make sure you filter the network down to just the FCodes used for VBET.
    4. Be aware of there potentially being multiple disconnected networks (i.e. multiple outflow places). Maybe we have a parameter that says each network much have at least 10 or 100 reaches etc before we bother generating a centerline.
  2. Generate a centerline from each starting point to its corresponding outflow point.
  3. The centerlines should merge and overlap at confluences. We need to remove duplicate line segments. The old code Matt wrote dealt with this issue.

Given that our VBET polygons are still flooding too much of the valley, perhaps this method should be developed standalone on a "clean" and "reliable" VBET generated by USU.

Here's the open source code for generating centerlines.

@philipbaileynar philipbaileynar added enhancement New feature or request pkg:VBET VBET python package labels Apr 7, 2021
@philipbaileynar philipbaileynar added this to the VBET Enhancements milestone Apr 7, 2021
@joewheaton joewheaton changed the title Valley centerline Valley-bottom centerline Jun 24, 2021
@joewheaton
Copy link
Contributor

So this is a duplicate of #270, but a great alternative strategy that @philipbaileynar suggests, but we never got around to trying.

@philipbaileynar
Copy link
Contributor Author

This ticket is old. Deassigning Kelly until we have a coherent strategy to move forward.

@KellyMWhitehead
Copy link
Contributor

Centerlines are getting generated with the new vbet.

@MattReimer MattReimer removed this from the Someday milestone Jan 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request pkg:VBET VBET python package
Projects
None yet
Development

No branches or pull requests

4 participants