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

Tolls for HGV considered only when vehicle_type is set #358

Closed
aoles opened this issue Nov 16, 2018 · 1 comment
Closed

Tolls for HGV considered only when vehicle_type is set #358

aoles opened this issue Nov 16, 2018 · 1 comment
Assignees
Labels
bug 🐞 Erroneous behavior of the backend
Milestone

Comments

@aoles
Copy link
Member

aoles commented Nov 16, 2018

At the moment in order to pick up hgv-specific tolls with avoid: tollways one has to additionally specify vehicle_type in options. Apart from the fact that this behavior us not documented, this is unpractical because currently the result doesn't depend on the particular value of vehicle_type anyway (even though it potentially could...).

Consider changing the logic such that results from TollwayExtractor depend on profile rather than vehicle_type.

@aoles aoles self-assigned this Nov 16, 2018
@aoles
Copy link
Member Author

aoles commented Nov 20, 2018

On top of this, for a given way only the first matching OSM toll tag is being processed. For example, if there is already toll=no than a more specific toll:hgv=yes might not be picked up, such as here: https://maps.openrouteservice.org/directions?n1=50.030584&n2=19.821138&n3=15&a=50.034209,19.815044,50.025704,19.823112&b=4a&c=0&f5=2&k1=en-US&k2=km

@aoles aoles added the bug 🐞 Erroneous behavior of the backend label Nov 21, 2018
@aoles aoles added this to the 4.7.2 milestone Nov 21, 2018
@aoles aoles mentioned this issue Dec 7, 2018
12 tasks
rabidllama pushed a commit that referenced this issue Dec 10, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐞 Erroneous behavior of the backend
Projects
None yet
Development

No branches or pull requests

2 participants