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

Lane suggestion rewrite #1572

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from
Draft

Conversation

Karry
Copy link
Collaborator

@Karry Karry commented May 17, 2024

Suggested route lanes are based on turn geometry now (DirectionDescription).

But based on my experience and extensive report Karry/osmscout-sailfish#326 , it provides misleading results on many junctions. So, it should analyze lanes and angles of all ways connected to the analyzed junction node...

Instead of evaluating route angle on junction,
it read lanes on the ALL outgoing junction ways
and use heuristics what lanes we can use to move forward.
Copy link

sonarcloud bot commented Jun 10, 2024

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature For issues and pull request that request or implement a new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants