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

routing with some turning restrictions is broken #1107

Closed
hlinden opened this issue Jun 30, 2014 · 8 comments
Closed

routing with some turning restrictions is broken #1107

hlinden opened this issue Jun 30, 2014 · 8 comments
Assignees

Comments

@hlinden
Copy link

hlinden commented Jun 30, 2014

http://osrm.at/8aw

leads to this route:

http://i.imgur.com/xxdPWal.png

...which of course is not correct. The other way round, the route is calculated as expected. There is no obvious error in the osm data, other routing solutions route correctly in both directions.

@DennisOSRM
Copy link
Collaborator

Thanks for reporting this. Seems to be an issue when a single road segment is the source of two is_only_* restrictions. Working on it.

@daganzdaanda
Copy link

Hi, i just happened to click around and came up with this: http://osrm.at/8Kd
Could be the same issue, or maybe not?

I think the tagging is correct, so at the node https://www.openstreetmap.org/node/289606224 one should be able tu turn left.
There is a no-u-turn relation, and a only_straight_on relation that may be playing into it.

ETA:
If you go the other way, it leads to this: http://osrm.at/8Kg
Weird, could not see anything in the data.

@daganzdaanda
Copy link

... I think this may be causing the hiccups: https://www.openstreetmap.org/relation/3483948
Compare http://osrm.at/8Kh and http://osrm.at/8Ki

If you switcht the direction, it looks as if it breaks at https://www.openstreetmap.org/relation/3483950

The OSM data seems to be OK, since Mapquest has no problem.

@DennisOSRM
Copy link
Collaborator

The issue has been identified. It will take a couple of days though until a patch is available.

@daganzdaanda
Copy link

cool, thanks!

@DennisOSRM
Copy link
Collaborator

Here's a quick update on the issue. And a picture says more than a thousand words:

bildschirmfoto 2014-08-07 um 18 27 31

Fix will arrive within the hour and the demo site should pull this in within the next 12--18 hours.

@DennisOSRM
Copy link
Collaborator

Fix is live on the demo site.

@daganzdaanda
Copy link

Thank you, great work!

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

No branches or pull requests

3 participants