search

north-road / qgis-processing-saga-nextgen

GNU General Public License v3.0
21 stars 11 forks source link

Raster loading error in tool #17

Open roya0045 opened 2 years ago

roya0045 commented 2 years ago

Greetings Nyall,

I am using the "Line to profile" tool and during the executing I get an error when SAGA is attempting to read the raster,

The raster is originally a Float64 as opposed to my other rasters, but now even with older rasters that were ok i get the error message when reading the raster input of ta_slopes

Loading grid: C:/Users/-/AppData/Local/Temp/processing_UKaRLq/9ca10f78d5924c078ffe4636f0461416/ATHDEM5mc.sgrd... failed Error: input file [C:/Users/-/AppData/Local/Temp/processing_UKaRLq/9ca10f78d5924c078ffe4636f0461416/ATHDEM5mc.sgrd] Error: DEM

Do you have a clue what might be causing this?

roya0045 commented 2 years ago

Greetings Nyall, I give it a try with hydrology tools and they all give the same error. The source is a tiff but when I do it directly in the saga app it works fine. I'd be willing into things to solve this if you could provide a guess on the difference between the two approaches and how to get closer to the native approach.

Thanks,

roya0045 commented 2 years ago

tried the command line without the io transform and all was fine. @nyalldawson Do you have any objection if I remove any use of https://github.com/north-road/qgis-processing-saga-nextgen/blob/5603227f7b42fd8729ddd8bed641621d340dd05c/processing_saga_nextgen/processing/SagaAlgorithm.py#L400 in the code? I think the transformed files get deleted faster than saga can use them and letting saga do the file handling itself seem to work fine for most formats.

roya0045 commented 2 years ago

via the command line the modified code is fine but not qgis, might be a gdal or other dependencies/path issues on my system... Though I still think the gdal_io is unneeded.