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

Style scenario network features to show if they are usable by foot and bicycle #1359

Closed
UJehle opened this issue Jun 28, 2022 · 1 comment
Closed
Assignees

Comments

@UJehle
Copy link
Contributor

UJehle commented Jun 28, 2022

Sometimes it happens, that the bridge scenario is not working properly and the "scenario" isochrone basically looks like the "standard" one. I think I now understood why: the network that the circle tool shows seems to be not in line with the routing network. At Wittelsbacherstraße for example.
Bild1

@UJehle UJehle added the Bug label Jun 28, 2022
@EPajares EPajares added this to the v1.2 milestone Jul 29, 2022
@EPajares EPajares self-assigned this Aug 5, 2022
@EPajares
Copy link
Collaborator

Many things are coming two things together here.

  1. First, the network on OSM is, from my point of view, wrongly classified. The paths are labeled as "use_sidepath", but no separate sidepaths are drawn. This is causing connectivity problems.

  2. We were fetching the whole network, including the parts that are labeled e.g. foot=use_sidepath or bicycle=use_sidepath.
    Though we are excluding these classes for the routing.

I am now filtering by the foot categories to exclude. Though this is not the perfect solution as we are not handling the ones excluded for bikes. The best would be styling the features to show whether they are usable by foot and bike. And show features that are not suitable for one or the other. Though this would require some more changes. So I have patched this temporary fix.

@EPajares EPajares changed the title routing network and fetched network with circle tool not in line? Style scenario network features to show if they are usable by foot and bicycle Aug 13, 2022
@EPajares EPajares added Front-end and removed Bug labels Aug 13, 2022
@EPajares EPajares removed this from the v1.2 milestone Aug 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants