search

nilsnolde / valhalla-qgis-plugin

QGIS plugin to use Valhalla for routing, isochrones and matrices.
GNU General Public License v3.0
12 stars 3 forks source link

Directions lines proc for truck is using pedestrian costing instead of truck costing #16

Closed radekvermirovsky closed 10 months ago

radekvermirovsky commented 10 months ago

This python script https://github.com/gis-ops/valhalla-qgis-plugin/blob/359f65a2ac897e66383d70c269c5bf2c06324dc3/valhalla/proc/directions_lines/directions_lines_truck.py

is using PedestrianCosting and IMO should use TruckCosting

https://github.com/gis-ops/valhalla-qgis-plugin/blob/359f65a2ac897e66383d70c269c5bf2c06324dc3/valhalla/proc/directions_lines/directions_lines_truck.py#L26

https://github.com/gis-ops/valhalla-qgis-plugin/blob/359f65a2ac897e66383d70c269c5bf2c06324dc3/valhalla/proc/directions_lines/directions_lines_truck.py#L36

nilsnolde commented 10 months ago

hehe yeah classic copy/paste fail

radekvermirovsky commented 10 months ago

Fixed in https://github.com/gis-ops/valhalla-qgis-plugin/pull/18