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

bad u-turn on road with turn restriction #2992

Closed
RedAuburn opened this issue Apr 28, 2024 · 5 comments
Closed

bad u-turn on road with turn restriction #2992

RedAuburn opened this issue Apr 28, 2024 · 5 comments

Comments

@RedAuburn
Copy link

Describe the bug
routing tries to make a u-turn to get to a road that has a turn restriction

To Reproduce
https://www.openstreetmap.org/directions?engine=graphhopper_car&route=51.47523%2C0.01003%3B51.47462%2C0.00983

Expected behaviour
should be routed the fastest legal way:
https://www.openstreetmap.org/directions?engine=fossgis_osrm_car&route=51.47523%2C0.01003%3B51.47462%2C0.00983

Screenshots

current / correct

image1

@AmyMagdalena
Copy link

AmyMagdalena commented Jun 4, 2024

Similar issue here, biking route makes illegal turn, rather than telling cyclists to dismount and use the pedestrian crossing, The car route functions correctly.

Road Tags:

bicycle=yes
highway=tertiary
horse=no
lanes=2
maxspeed=30
motor_vehicle=yes
name=Carrer Gran de Gràcia
name:ca=Carrer Gran de Gràcia
name:etymology:wikidata=Q852697
oneway=yes
restriction=no_left_turn
surface=asphalt
wikidata=Q1805479
wikipedia=ca:Carrer Gran de Gràcia

https://www.openstreetmap.org/directions?engine=graphhopper_bicycle&route=41.40466%2C2.15116%3B41.40537%2C2.15000#map=19/41.40539/2.15064
**
imatge
**

@easbar
Copy link
Member

easbar commented Jun 4, 2024

routing tries to make a u-turn to get to a road that has a turn restriction

Yes, this is because the penalty for making the u-turn is too low, related: #2883

Similar issue here, biking route makes illegal turn, rather than telling cyclists to dismount and use the pedestrian crossing,

The eastern crossing used to be tagged as bicycle=no until a few hours ago so it is still ignored for bikes. The other issue here is that turn restrictions aren't considered for bicycles, yet.

@AmyMagdalena
Copy link

Yes, I fixed that and am aware the cache takes time to update, I just don't think it'll ever choose to dismount over making the illegal turn. As you say, it's not considered.

@easbar
Copy link
Member

easbar commented Jun 4, 2024

I just don't think it'll ever choose to dismount over making the illegal turn.

Yes, currently it won't, because for bicycle=dismount there is also a penalty. But once we enable turn restrictions for bicycles the route with the road crossing should be preferred.

@karussell
Copy link
Member

karussell commented Aug 27, 2024

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

4 participants