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

Total route duration !== sum of all maneuver durations #1575

Closed
peterqliu opened this issue Jul 22, 2015 · 3 comments
Closed

Total route duration !== sum of all maneuver durations #1575

peterqliu opened this issue Jul 22, 2015 · 3 comments

Comments

@peterqliu
Copy link

http://osrm.at/dLQ

This trip has a total duration of 95s, but its first maneuver takes 169s.

cc/ @lbud

@TheMarex
Copy link
Member

TheMarex commented Aug 1, 2015

Best guess would a bug in the edge compression. Note that there should be a turn announced.

@TheMarex
Copy link
Member

TheMarex commented Aug 3, 2015

Okay found the root cause of this:

Path unpacking is broken for the first and last segment if these segments are compressed edges. In that case PathData will contain the full length of the compressed edge - not only the length of the part that is covered by the PhantomNode.

This has some bigger implications on the query code and description generation code. Currently working on fixing this and simplifying the code along the way.

@danpat
Copy link
Member

danpat commented Aug 26, 2016

This has been fixed.

Current API request:

https://router.project-osrm.org/route/v1/driving/-122.98851,37.995975;-122.998009,37.99736?overview=false&alternatives=true&steps=true

sum of steps durations == summary duration.

@danpat danpat closed this as completed Aug 26, 2016
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