Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Continuous lane guidance fix #3054

Merged
merged 8 commits into from
May 6, 2021
Merged

Continuous lane guidance fix #3054

merged 8 commits into from
May 6, 2021

Conversation

dgearhart
Copy link
Member

@dgearhart dgearhart commented May 4, 2021

Issue

Fix the continuous lane guidance logic to properly activate turn lanes when the next maneuver type is stay left or stay right.

Example#1

Before vs. After diff

image

The continuous active lanes are now fixed to match the path

image


Example#2

Before vs. After diff

image

Suggests that the user should be in the left most slight_right lane to be ready to keep left at the fork in the ramp

image


This update improved 23% of user routes

Tasklist

  • Add tests
  • Add #fixes with the issue number that this PR addresses
  • Update the changelog

@dgearhart dgearhart requested a review from gknisely May 5, 2021 18:33
@dgearhart dgearhart merged commit 1d379cc into master May 6, 2021
@dgearhart dgearhart deleted the continuous_lane_fix branch January 5, 2022 16:29
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants