-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
Counting of service exits on service roundabouts #4075
Comments
with the current work on roundabouts, keep in mind that roundabout exit instructions are kind of broken for cyclists and pedestrians, eg. #2533. you can of course use serviceroads, and also push against oneways. here's an example of a roundabout with highway=path on both the roundabout and all exists: and a roundabout with a cycleway exit: |
here's a roundabout with one residential exit, two service roads, and a cycleway: |
@emiltin thanks for pointing to the issue for non-car profiles! i will check in scope of this issue, but ignoring service roads on normal roundabouts atm is expected behavior for car profiles. Here http://map.project-osrm.org/?z=19¢er=55.739516%2C12.494368&loc=55.739479%2C12.494186&loc=55.739648%2C12.494454&hl=en&alt=0 "the 1st exit" is correct. Maybe a special handling for cyclists and pedestrians is needed, but i would open an issue after a check. |
yes my point is that the current behaviour does not work for foot and bike routing. |
OSM has 13150 ways
["highway"="service"]["junction"="roundabout"];
in which service exits will be suppressed and guidance instructions will be"take the 1st exit"
. Most of the service roundabouts are located in large parking areas like airports or hospitals and guidance can be confusing in these areas.For http://map.project-osrm.org/?z=18¢er=40.479392%2C-3.477795&loc=40.478358%2C-3.479485&loc=40.480883%2C-3.477607&hl=en&alt=0
the third instruction must be "take the 2bd exit"
In #4073 commit 588563c changed the exits counting, but not related to the PR.
The text was updated successfully, but these errors were encountered: