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

"Merge" icon still reversed #802

Closed
ericrwolfe opened this issue Nov 6, 2017 · 4 comments
Closed

"Merge" icon still reversed #802

ericrwolfe opened this issue Nov 6, 2017 · 4 comments

Comments

@ericrwolfe
Copy link
Contributor

It looks like the merge icon is still reversed. This was a "merge left" maneuver but the icon shows merge right:

img_0224

cc @frederoni

@frederoni
Copy link
Contributor

frederoni commented Nov 7, 2017

This particular merge is a straight merge:

bearing_after:161
bearing_before:153
modifier:"straight"

so we end up not flipping the icon because it's not left, slight left, or sharp left.

Merge, take off ramp, and reach fork could use a different logic where the slightest difference in bearing should determine the direction.

@ericrwolfe
Copy link
Contributor Author

For merge straight (or any straight modifiers) we should probably just show a straight arrow.

@1ec5
Copy link
Contributor

1ec5 commented Nov 8, 2017

When we say “merge left”, what that means is that there’s traffic coming in over the user’s left shoulder, not that the user needs to turn left at the maneuver. Since we aren’t having users literally fall off ramps into traffic below, there should be no such thing as a “straight merge”. This is almost certainly a regression in the Directions API. I’ve filed a bug report upstream in that project.

/ref Project-OSRM/osrm-text-instructions#183

@1ec5
Copy link
Contributor

1ec5 commented Nov 10, 2017

Fixed upstream, server-side, in Project-OSRM/osrm-backend#4670.

@1ec5 1ec5 closed this as completed Nov 10, 2017
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

3 participants