Skip to content
This repository has been archived by the owner on Dec 16, 2020. It is now read-only.

Matrix API bug for disconnected locations #40

Closed
karussell opened this issue Sep 29, 2016 · 1 comment
Closed

Matrix API bug for disconnected locations #40

karussell opened this issue Sep 29, 2016 · 1 comment

Comments

@karussell
Copy link
Member

karussell commented Sep 29, 2016

curl "https://graphhopper.com/api/1/matrix?point=55.925644%2C-5.152433&point=55.918959%2C-5.159128&type=json&vehicle=small_truck&debug=true&out_array=weights&out_array=times&out_array=distances&key=[key]"

This returns time and distance == 0 but should either throw a RuntimeException (would be consistent with the Routing API) or an 'infinite' value to still use parts of the results.

Although we normally prefer fail fast, I would prefer an 'infinite' value here. Some other Matrix API providers return negative values which I don't like too.

@karussell
Copy link
Member Author

karussell commented Sep 29, 2016

Fixed via explicitely throwing the RuntimeException (keep consistency with Routing API). This will be deployed in the next ~30h

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant