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 does not support :forward and :backward modifiers on access keys #8507

Open
eisams opened this issue Jun 20, 2024 · 9 comments
Open
Labels
Routing Route building issues, e.g. valid route, valid ETA

Comments

@eisams
Copy link
Contributor

eisams commented Jun 20, 2024

Organic Maps will happily route you through forbidden routes, this is not allowed:
Screenshot 2024-06-20 at 21 03 30

See how Valhalla does it: https://www.openstreetmap.org/directions?engine=fossgis_valhalla_car&route=59.91734%2C10.75949%3B59.91877%2C10.75936

These are generic access modifiers that can apply to any normal access tag e.g., vehicle, motor_vehicle and bus
Screenshot 2024-06-20 at 21 06 14

The specific way in question: https://www.openstreetmap.org/way/17844#map=19/59.91809/10.75935

(The Valhalla routing will change at the next update, as it routes through a set of roads that I corrected destination only access)

@biodranik
Copy link
Member

Looks like motor_vehicle:forward=no is ignored, right @vng ?

@vng
Copy link
Member

vng commented Jun 20, 2024

908 occurrences, https://taginfo.openstreetmap.org/keys/motor_vehicle%3Aforward#overview
Not mentioned here:
https://wiki.openstreetmap.org/wiki/Key:motor_vehicle
I suspect it is not a valid mapping. Is it the same as putting oneway=-1 ?

@matkoniecz
Copy link
Contributor

I created https://www.openstreetmap.org/note/4301123

Not mentioned here:

well, :forward and :backward prefixes can be added to anything but this usage seems like a pointless overcomplication and trap

http://overpass-turbo.eu/s/1Na6

though you have say https://www.openstreetmap.org/way/1271671965 with motor_vehicle:forward=destination that has no obvious replacement

@harahu
Copy link

harahu commented Jun 21, 2024

Tricky this one. The street is oneway, but only for motor_vehicles, and even then with two exceptions:

  • Busses are allowed to drive this direction when servicing a route.
  • Goods transport is allowed.

In light of this I feel like a blanket oneway=yes would be misleading unless one can specify the exceptions well enough in a way that most routers understand.

@matkoniecz
Copy link
Contributor

I expect that no tagging will work well in most routers as case is quite rare.

@harahu
Copy link

harahu commented Jun 21, 2024

I am digging into the documentation on these kinds of restrictions. Some observations:

@harahu
Copy link

harahu commented Jun 21, 2024

I guess you mean this: valhalla/valhalla#4203

@eisams
Copy link
Contributor Author

eisams commented Jun 21, 2024

I deleted my ugly replies by email, just wanted to say that it is implemented by Grasshopper, Valhalla and OSRM
Valhalla implemented it after I reported it last year

The relevant tags for the routing modes supported in OM:
https://taginfo.openstreetmap.org/keys/motor_vehicle%3Aforward#overview (900)
https://taginfo.openstreetmap.org/keys/vehicle%3Aforward#overview (800)
https://taginfo.openstreetmap.org/keys/bicycle%3Aforward#overview (6000)

https://taginfo.openstreetmap.org/keys/motor_vehicle%3Abackward#overview (1200)
https://taginfo.openstreetmap.org/keys/vehicle%3Abackward#overview (800)
https://taginfo.openstreetmap.org/keys/bicycle%3Abackward#overview (6700)

@patepelo patepelo added the Routing Route building issues, e.g. valid route, valid ETA label Jun 22, 2024
@eisams
Copy link
Contributor Author

eisams commented Jul 1, 2024

FYI: Just checked and OsmAnd also support these tags, and even shows it as one-ways. (albeit with a very interesting route, didn't know you could take that shortcut)

Screenshot 2024-07-01 at 19 18 27

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Routing Route building issues, e.g. valid route, valid ETA
Projects
None yet
Development

No branches or pull requests

6 participants