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

Return snapped coordinate in distance table plugin #1637

Closed
peterqliu opened this issue Sep 1, 2015 · 2 comments
Closed

Return snapped coordinate in distance table plugin #1637

peterqliu opened this issue Sep 1, 2015 · 2 comments

Comments

@peterqliu
Copy link

If I request driving directions from A to B, with C being the nearest routable neighbor to B, the reported duration is really of A to C. If B and C are very distant from each other, the duration prediction can be way off.

This issue becomes more significant when the new Distances API doesn't report C at all, making it difficult to gauge just how near the nearest neighbor is, and, by extension, how reliable the estimate is.

I don't have an ideal solution, but something that may work better would account for walking from and to the origin and destination, respectively.

@TheMarex
Copy link
Member

TheMarex commented Sep 1, 2015

You mean, you don't know which coordinate the nearest neighbor search snapped to, thus making it hard to debug the travel time given in the matrix? Very good point.

We should add this to the response of the table plugin - only problem here is that we currently add all the candidates to the heap and never check which one is used.

Also related to #1447

@TheMarex TheMarex reopened this Sep 1, 2015
@TheMarex TheMarex changed the title Nearest neighbor can misleading for duration estimates Return snapped coordinate in distance table plugin Sep 1, 2015
@TheMarex
Copy link
Member

This was implemented in our v5 API, we now return the snapped location of every input coordinate.

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

2 participants