search

nextzen / lrm-mapzen

Support for Mapzen Turn-by-Turn routing in Leaflet Routing Machine
https://mapzen.com/projects/turn-by-turn/
Other
86 stars 31 forks source link

Problem with pedestrian mode #55

Closed sylvainar closed 8 years ago

sylvainar commented 8 years ago

Hey guys !

Nice work first ;)

So i'm having an issue with LRM and Mapzen, my website is able to calculate the routing in auto, bicycle mode, but not in the only mode I really need, pedestrian.

This is the code I'm using : var routing = L.Routing.control({ router: L.Routing.mapzen('valhalla-TC63npX', {costing:'pedestrian'}), formatter: new L.Routing.mapzenFormatter(), waypoints: routing_poi_list, routeWhileDragging: false, collapsible: true, draggableWaypoints: false, addWaypoints: false }).addTo(map);

And the error (sorry it's in french) :
Error : No path could be found for input lrm-mapzen.min.js:1:3757 erreur de syntaxe

No path could be found : I don't understand why because it works in other modes. Syntax error ? Why ?

Thanks for your help !

hanbyul-here commented 8 years ago

Hi!

You can't get pedestrian routing result at all? or is this happening only with some waypoints?

sylvainar commented 8 years ago

Hey,

Nop, only with some waypoints, but it doesn't work, like, 8 on 10 tries. Here are some screenshots in Paris.

capture du 2016-05-27 18 30 41 capture du 2016-05-27 18 30 00

kevinkreiser commented 8 years ago

@sylvainar the problem isnt lrm but rather with the valhalla service code. particularly, the problem is that there are really short ways near by that are marked as stairs. pedestrians are allowed on stairs and so the algorithm finds those as the starting point. the problem is the stairs arent connected to anything. so once the algorithm is on the stairs it cant get off.

we are tracking this issue here: https://github.com/valhalla/loki/issues/127

sylvainar commented 8 years ago

Great, I suscribe to it. Thanks !