gis-ops / 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

Proper support for Polygons and MultiPolygons #20

Closed chrstnbwnkl closed 4 months ago

chrstnbwnkl commented 4 months ago

With https://github.com/valhalla/valhalla/pull/4575 merged, this makes the plugin compatible with that latest update to the isochrone contour generation. Should be largely backwards compatible to older versions of Valhalla, I've found out a lot of renderers don't really care for correct ring grouping anyways, which is the only thing that might break (before, the plugin would just ignore any polygon rings other than the first one). I'd say let's wait and see if someone complains :)

chrstnbwnkl commented 4 months ago

Yeah would be more readable but this plugin is really in maintenance mode so I thought the less changes the better haha