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

Detect Difference between On/Off-ramps when looking at forks in motorways #3257

Closed
MoKob opened this issue Nov 8, 2016 · 4 comments
Closed
Labels

Comments

@MoKob
Copy link

MoKob commented Nov 8, 2016

At this location,

screen shot 2016-11-08 at 09 57 46

we are currently emitting a fork instruction. Based from how the intersection looks, it might be better to emit an exit for the turn to the right and no instruction for the turn to the left.

We could think about looking ahead on forks in motorways to see if the next intersection is a merge onto a motorway or an exit. If we see a normal intersection, we could switch to exit (could be confusing if we told someone to use the ramp before?) and if we see a motorway on the other side, we could include that fact in making the turn to stay on the motorway the obvious one.

Question here: is staying on the motorway always more obvious than taking an exit?

@MoKob MoKob added the Guidance label Nov 8, 2016
@daniel-j-h daniel-j-h added this to the 5.5.0 milestone Nov 8, 2016
@willwhite
Copy link
Contributor

Another heuristic might be lane information (if available). This example has two lanes to the left and one to the right. Maybe that situation could always be treated like an exit/ramp?

@MoKob
Copy link
Author

MoKob commented Nov 9, 2016

@willwhite looking at the picture, that is exactly what I thought as well. @chaupow you could use lanes to better distinguish forks.

However, this would only result in a turn right. We still would need some additional heuristic to detect the exit part here. (e.g. we could check for available exit numbers?)

@TheMarex TheMarex modified the milestones: 5.6.0, 5.5.0 Nov 14, 2016
@TheMarex TheMarex modified the milestones: 5.7.0, 5.6.0 Jan 28, 2017
@TheMarex TheMarex removed this from the 5.7.0 milestone Mar 20, 2017
@daniel-j-h
Copy link
Member

Can be worked around on the client side: if there' exit information always announce as an exit:

Project-OSRM/osrm-text-instructions#128

@MoKob
Copy link
Author

MoKob commented Jul 27, 2017

This is a duplicate of #4295 .

@MoKob MoKob marked this as a duplicate of #4295 Jul 27, 2017
@MoKob MoKob closed this as completed Jul 27, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants