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.

Sign up for GitHub

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

Jump to bottom

Remove nodes that are likely not part of the network #161

Closed

barneydobson opened this issue May 13, 2024 · 1 comment

Labels

Collaborator

barneydobson commented May 13, 2024 •

edited

Loading

We might not want all pedestrian networks to be considered, as this will potentially put a lot of pipes and manholes in dumb locations. The only simple solution I can think of would be to add some weight parameter to these networks to avoid them if possible, but it doesn't feel very satisfying. cheginit - wonder if you have any thoughts on this?

Originally posted in #143 by @barneydobson

I haven't tested how this interacts with the community approach introduced in #144 , possibly some cleverness around that could also help

Pedestrian network could have a stronger merge perhaps - it seems even more important for these networks than the main network.

barneydobson added the enhancements label

barneydobson mentioned this issue

Pedestrian pipes #143

Merged

barneydobson added the on hold label

Collaborator Author

barneydobson commented Jun 13, 2024

Fixed by #143 and #144

barneydobson closed this as completed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment