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

Fix:xml:Fix bike routing #850

Merged
merged 2 commits into from
Sep 1, 2019
Merged

Fix:xml:Fix bike routing #850

merged 2 commits into from
Sep 1, 2019

Conversation

mvglasow
Copy link
Contributor

This improves bike routing (see also #844):

  • Oneway restrictions are respected for bikes (if a way has different oneway restrictions for bikes, create two ways, one for each direction, and set access flags accordingly)
  • Trunk roads are allowed if access flags permit, but with a low priority
  • Unpaved paths get a low speed and priority so paved paths are preferred
  • Priority decreases as road order increases

Tests with the use cases in #844 show significant improvements over the previous situation. There may not be a perfect solution because:

  • We do not have elevation information so we cannot consider that for routing.
  • We do know if a way where both pedestrians and bikes are allowed is segregated (we would prefer segregated ways for routing), or if a designated bike path runs right next to a main road (which some bikers might not like)
  • Finding an easy-to-follow route is not always easy as most road networks are structured primarily with cars in mind, not medium-distance biking (and we do not have medium/long-distance bike path information in the map data, so we cannot consider that)

All of these would require a different approach to routing, primarily a cost function which does not solely depend on vehicle profile, item type and distance.

Respect oneway restrictions
Allow trunk, but with low priority
Lower priority for unpaved paths
Tweak speed and priority for other item types

Signed-off-by: mvglasow <michael -at- vonglasow.com>
@mvglasow
Copy link
Contributor Author

Fixes #844.

@jkoan jkoan added this to the 0.5.4 milestone Aug 30, 2019
@mvglasow mvglasow merged commit 94834b7 into navit-gps:trunk Sep 1, 2019
viktorgino pushed a commit that referenced this pull request Sep 22, 2020
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

Successfully merging this pull request may close these issues.

2 participants