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

Proposed ways are included in routing for bicycle and foot profiles #6614

Open
lliehu opened this issue Apr 24, 2023 · 0 comments
Open

Proposed ways are included in routing for bicycle and foot profiles #6614

lliehu opened this issue Apr 24, 2023 · 0 comments

Comments

@lliehu
Copy link
Contributor

lliehu commented Apr 24, 2023

Issue

According to OpenStreetMap wiki, ways with highway=proposed and proposed=* tags are for "any feature that is still in planning phase but construction hasn't yet started". These ways are currently avoided in car profile but bicycle and foot profiles happily route on these ways even though it's possible that nothing exists on the ground.

It looks like these ways were intentionally discarded for routing on all of these profiles (car, bicycle and foot) on Jul 10, 2017. Link to merged pull request: #4258

Interestingly, routing on proposed ways was effectively re-enabled for bicycle and foot profiles about a week later on Jul 18 when a large pull request #4072 was merged. Reading through the pull request, there were no comments nor any mention in the commit message about this change. Note that the behavior was retained for car profile that had tests. It seems possible that the behavior was accidentally lost for bicycle and foot profiles during a rebase for this large pull request because these profiles did not have any tests for this behavior. taginfo.json also still says that both construction and proposed ways are discarded for routing and doesn't mention that it only applies to certain profiles.

Steps to reproduce

At first, I noticed this problem when openstreetmap.org routed me through the following proposed way: https://www.openstreetmap.org/way/386217696

Results of further investigation are written above. Cucumber tests and a possible fix are included in pull request #6615.

Specifications

  • osrm-backend from latest master branch
  • clang version 15.0.7 (failed to build with gcc 12)
  • Arch Linux updated less than a day ago
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

1 participant