[openstreetmap/openstreetmap-website] Include car profile for GraphHopper routing (#1547)

[Tom Hughes]

tomhughes commented on this pull request.



> @@ -79,5 +80,6 @@ function GraphHopperEngine(id, vehicleType) {
   };
 }
 
-OSM.Directions.addEngine(new GraphHopperEngine("graphhopper_bicycle", "bike"), true);
-OSM.Directions.addEngine(new GraphHopperEngine("graphhopper_foot", "foot"), true);
+OSM.Directions.addEngine(new GraphHopperEngine("graphhopper_car", "car", true), true);

I assume that `disableCH` refers to contraction hierarchies, in which case I would have thought long distance car routing would be exactly the case where you would want them so I'm surprised that you are disabling them for car routing but leaving them enabled for other things?

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/1547#pullrequestreview-39592647
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20170522/8441cd7f/attachment.html>