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

Notifications for Tunnels, Toll stations, etc. #3028

Open
daniel-j-h opened this issue Oct 10, 2016 · 3 comments
Open

Notifications for Tunnels, Toll stations, etc. #3028

daniel-j-h opened this issue Oct 10, 2016 · 3 comments
Assignees

Comments

@daniel-j-h
Copy link
Member

At the moment we're using Notification for travel mode changes (e.g. driving, ferry). It makes sense to convey more semantic meaning to clients for example for (longer) tunnels or other road network features.

Example:

tunnel

http://map.project-osrm.org/?z=16&center=51.046298%2C13.697033&loc=51.051080%2C13.705080&loc=51.047478%2C13.687710&hl=en&alt=0

At the moment the workaround here is to parse the name field and hope that longer tunnels have a dedicated name. But what about e.g. toll stations, parking lots, etc.?

What's our roadmap with regard to that?

@daniel-j-h daniel-j-h added this to the 6.0 milestone Oct 10, 2016
@MoKob
Copy link

MoKob commented Oct 11, 2016

Right now, I would say that mode change is the wrong paradigm to use in tunnels/bridges/toll roads. After all, you are still travelling by driving the car.

Mode-Changes are used to describe actual changes in mode like pushing your bike, instead of riding it or taking the ferry instead of driving.

We can use notification to emit informations like tunnels, but we would need some form of annotation feature. The hope part you mention is more like a shortcoming which we cannot handle so far. The continue instructions for a tunnel are only due to the new names occurring. Ideally we wouldn't want to mention an intermediate tunnel by name as an instruction.

@daniel-j-h
Copy link
Member Author

Update: with 5.9 we have the road classes feature in place and can easily add a tunnel road class if needs be. We'd need to parse the tunnel= tag and set the road class in the way handler.

Copy link

github-actions bot commented Jul 8, 2024

This issue seems to be stale. It will be closed in 30 days if no further activity occurs.

@github-actions github-actions bot added the Stale label Jul 8, 2024
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