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

Add support for Mapzen's Valhalla routing engine #1064

Merged
merged 1 commit into from
Sep 30, 2015
Merged

Add support for Mapzen's Valhalla routing engine #1064

merged 1 commit into from
Sep 30, 2015

Conversation

tomhughes
Copy link
Member

No description provided.

@kevinkreiser
Copy link

looks great! thanks so much for doing this! :shipit:

@dgearhart
Copy link

thank you 👍

@gknisely
Copy link

:shipit: thanks!

@openstreetmap-mirror openstreetmap-mirror merged commit e54eade into openstreetmap:master Sep 30, 2015
@tomhughes tomhughes deleted the mapzen branch September 30, 2015 13:06
@dgearhart
Copy link

I am not seeing Mapzen in the direction engine pull down anymore

@ppawel
Copy link
Contributor

ppawel commented Sep 30, 2015

Car (Mapzen) returns a lot of 400 Bad Request "No path could be found for input" responses (which is quite a silly response code because there is nothing wrong with a request for a route between two largest cities in Poland and OSRM finds it properly anyway) and the result on the website is that the loading spinner is shown indefinitely - shouldn't there be a "error ocurred" or "couldn't find route" message?

@kevinkreiser
Copy link

indeed if it cant find a route, the body of the 400 will say as much. can you give the locations in the route so we can look into it?

@tomhughes
Copy link
Member Author

Guys, can we stop using this bug as a dumping ground for every routing issue please...

@openstreetmap openstreetmap locked and limited conversation to collaborators Sep 30, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants