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

barrier=height_restriction should not prevent routing #4823

Closed
osrmuser opened this issue Jan 23, 2018 · 2 comments
Closed

barrier=height_restriction should not prevent routing #4823

osrmuser opened this issue Jan 23, 2018 · 2 comments

Comments

@osrmuser
Copy link

I think I have found a problem related to the handling of barrier=height_restriction on http://map.project-osrm.org/?z=18&center=52.140220%2C11.631485&loc=52.139836%2C11.630348&loc=52.139675%2C11.631544&hl=en&alt=0. It looks like OSRM avoids node https://www.openstreetmap.org/node/3247447825, probably beacause it is handled as an unpassable barrier without something like an explicit access=yes. But https://wiki.openstreetmap.org/wiki/Tag%3Abarrier%3Dheight_restrictor does not mention any explicit acces rule as required.

Since I have found a barrier whitelist in https://github.com/Project-OSRM/osrm-backend/blob/master/profiles/car.lua#L49, which seems to take care of similar problems, height_restriction probably should be added there (and maybe in other profiles too).

@MichalPP
Copy link
Contributor

maybe it's time to switch from barrier_whitelist to barrier_blacklist #3890

MichalPP added a commit to MichalPP/osrm-backend that referenced this issue May 12, 2018
change barrier_whitelist to barrier blacklist  in foot profile
fix Project-OSRM#5067 and Project-OSRM#3890 and Project-OSRM#4823
TheMarex pushed a commit that referenced this issue May 29, 2018
change barrier_whitelist to barrier blacklist  in foot profile
fix #5067 and #3890 and #4823
@osrmuser
Copy link
Author

Verified fixed, thx MichalPP and TheMarex

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

2 participants