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

Routing through motor_vehicle=destination #501

Closed
gomyhr opened this issue Nov 11, 2012 · 7 comments
Closed

Routing through motor_vehicle=destination #501

gomyhr opened this issue Nov 11, 2012 · 7 comments

Comments

@gomyhr
Copy link

gomyhr commented Nov 11, 2012

OSRM is happy to route through Buengveien (http://www.openstreetmap.org/?lat=63.368388&lon=10.376653&zoom=18) which is marked with motor_vehicle=destination.

@emiltin
Copy link
Contributor

emiltin commented Nov 11, 2012

access=destination is not implemented yet in osrm. but i don't think motor_access is a valid osm tag? http://wiki.openstreetmap.org/wiki/Key:access

@gomyhr
Copy link
Author

gomyhr commented Nov 11, 2012

Sorry, it's motor_vehicle=destination. I updated the description above. I thought destination was implemented, since I saw it mentioned the beginning of profile.lua:
access_tag_restricted = { ["destination"] = true, ["delivery"] = true }
(https://github.com/DennisOSRM/Project-OSRM/blob/e0f51dd5e9e67f52e32d585f966853ab562554a4/profile.lua)

@emiltin
Copy link
Contributor

emiltin commented Nov 11, 2012

if you run the relevant cucumber tests, you'll see that they fail:

cucumber --tag @destination --tag @car

@emiltin
Copy link
Contributor

emiltin commented Jan 29, 2013

not yet implemented, but closing this for now, since it's not a bug.

@emiltin emiltin closed this as completed Jan 29, 2013
@naoliv
Copy link

naoliv commented Jan 29, 2013

Shouldn't wishlist bugs stay open to avoid possible duplicate bug reports and also to indicate that it will be handled/fixed/implemented in future?
Leaving it open also works as a TODO list (or roadmap): people can see what is possibly planned for a future release.

@emiltin
Copy link
Contributor

emiltin commented Jan 29, 2013

you're right.
there's already an issuse covering this at #156, so keeping this one clsoed.

@naoliv
Copy link

naoliv commented Jan 29, 2013

#156, got it! Thanks!
:-)

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

3 participants