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

Fix processing and revert plugin folder name change #13

Closed nilsnolde closed 9 months ago

nilsnolde commented 9 months ago

fixes #12

And unfortunately the QGIS plugin system is a bit dumb when it comes to plugin name/folder structure. It has to be called valhalla, same as the plugin name. The awkward thing is that this clashes obviously with https://github.com/gis-ops/pyvalhalla which also expects to be imported as valhalla. Only way forward seems to be to rename this plugin. Anyways, we'll simply remove this plugin altogether once we (finally) release the network analyst plugin.

florisvdh commented 9 months ago

Great, thanks for the prompt response!

I can confirm that it works:

$ qgis_process plugins
Available plugins
(* indicates loaded plugins which implement Processing providers)

* QNEAT3
  valhalla
* processing_saga_nextgen
* pcraster_tools
* ViewshedAnalysis
* cartography_tools
* wbt_for_qgis
* latlontools
* grassprovider
* otbprovider
* processing
$ 
$ qgis_process plugins enable valhalla
Enabling plugin: "valhalla"
Enabled valhalla (Valhalla)

Available plugins
(* indicates enabled plugins which implement Processing providers)

* QNEAT3
* valhalla
* processing_saga_nextgen
* pcraster_tools
* ViewshedAnalysis
* cartography_tools
* wbt_for_qgis
* latlontools
* grassprovider
* otbprovider
* processing
$ 
$ qgis_process plugins
Available plugins
(* indicates loaded plugins which implement Processing providers)

* QNEAT3
* valhalla
* processing_saga_nextgen
* pcraster_tools
* ViewshedAnalysis
* cartography_tools
* wbt_for_qgis
* latlontools
* grassprovider
* otbprovider
* processing
$ 
$ qgis_process plugins disable valhalla
Disabling plugin: "valhalla"
Disabled valhalla

Available plugins
(* indicates enabled plugins which implement Processing providers)

* QNEAT3
  valhalla
* processing_saga_nextgen
* pcraster_tools
* ViewshedAnalysis
* cartography_tools
* wbt_for_qgis
* latlontools
* grassprovider
* otbprovider
* processing
$ 
$ qgis_process plugins enable valhalla
Enabling plugin: "valhalla"
Enabled valhalla (Valhalla)

Available plugins
(* indicates enabled plugins which implement Processing providers)

* QNEAT3
* valhalla
* processing_saga_nextgen
* pcraster_tools
* ViewshedAnalysis
* cartography_tools
* wbt_for_qgis
* latlontools
* grassprovider
* otbprovider
* processing
$ 
$ qgis_process --version
QGIS 3.32.3-Lima 'Lima' (67d46100b5)
QGIS code revision 67d46100b5
Qt version 5.15.3
Python version 3.10.12
GDAL/OGR version 3.6.4
PROJ version 9.1.1
EPSG Registry database version v10.076 (2022-08-31)
GEOS version 3.11.1-CAPI-1.17.1
SQLite version 3.37.2
OS Linux Mint 21.2