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

Failing to collapse sliproad #4577

Closed
willwhite opened this issue Oct 5, 2017 · 3 comments
Closed

Failing to collapse sliproad #4577

willwhite opened this issue Oct 5, 2017 · 3 comments

Comments

@willwhite
Copy link
Contributor

Found another case where OSRM is failing to collapse a slip road here:

https://www.mapbox.com/get-directions/#17.36/37.29941/-121.83948?coordinates=-121.83803529740746,37.299429295255706;-121.83948370888538,37.29940937280185

@miccolis
Copy link
Contributor

miccolis commented Oct 5, 2017

@oxidase or @MoKob can you tell it this related to scenario 1 (the only remaining open one) on #4348 ?

@oxidase
Copy link
Contributor

oxidase commented Oct 6, 2017

@miccolis atm the issue is not fixed in master. The problem is that sliproad obviousness check fails at

// perfectly straight turns prevent obviousness
const auto turn_deviation = angularDeviation(intersection[i].angle, STRAIGHT_ANGLE);
if (turn_deviation < FUZZY_ANGLE_DIFFERENCE)
return 0;
with

(gdb) p intersection[i].angle
$17 = 158.95050803358333
(gdb) p turn_deviation
$18 = 21.049491966416667
(gdb) p FUZZY_ANGLE_DIFFERENCE
$19 = 25

It is fixed in fix/obvious-2 branch of #4426
screenshot from 2017-10-06 17-17-01

@oxidase oxidase added this to the 5.13.0 milestone Oct 6, 2017
@TheMarex TheMarex removed this from the 5.13.0 milestone Dec 18, 2017
@oxidase
Copy link
Contributor

oxidase commented Apr 25, 2018

Fixed in 5.17.0
screenshot from 2018-04-25 15-30-10

@oxidase oxidase closed this as completed Apr 25, 2018
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

4 participants