Re: [Qgis-developer] Yet again: SAGA 2.3.1 fails

2016-10-24 Thread Giovanni Manghi
Hi,

> Il 24/10/2016 10:32, Giovanni Manghi ha scritto:
>
>> attached). I have used it a few times in the past to change/create the
>> descriptions files when a new SAGA version was published.
>
> could you please do a PR instead of keeping them local?

the script? weird, I'm pretty sure I got it from you.

About the diff files with parameters differences (and new tools)
between different SAGA versions: I don't have anything locally. In the
recent past when I had more free time I created such files and made
the proper PRs, an example:

https://github.com/qgis/QGIS/commit/f8ea9d926872fd36ad06890a48308b5a8eea8078

anyway now I downloaded the list of most recent SAGA versions cmd
tools/parameters from here

https://sourceforge.net/p/saga-gis/wiki/SAGA%20Release%20Information/

and create such files, that I'm attaching here.

cheers

-- G --


saga_diffs.tar.gz
Description: GNU Zip compressed data
___
Qgis-developer mailing list
Qgis-developer@lists.osgeo.org
List info: http://lists.osgeo.org/mailman/listinfo/qgis-developer
Unsubscribe: http://lists.osgeo.org/mailman/listinfo/qgis-developer

Re: [Qgis-developer] Yet again: SAGA 2.3.1 fails

2016-10-24 Thread Paolo Cavallini
Il 24/10/2016 10:44, Richard Duivenvoorde ha scritto:

> Mmm, I do not think Giovanni is to blame.

sorry, too fast - I did not mean to blame Giovanni.

> Better to use this script to create a test with earlier output as a
> fixture to retest against?

yes; in principle supporting saga 2.3 LTR should fix this for a while.
All the best, and thanks.

-- 
Paolo Cavallini - www.faunalia.eu
QGIS & PostGIS courses: http://www.faunalia.eu/training.html
https://www.google.com/trends/explore?date=all=IT=qgis,arcgis
___
Qgis-developer mailing list
Qgis-developer@lists.osgeo.org
List info: http://lists.osgeo.org/mailman/listinfo/qgis-developer
Unsubscribe: http://lists.osgeo.org/mailman/listinfo/qgis-developer

Re: [Qgis-developer] Yet again: SAGA 2.3.1 fails

2016-10-24 Thread Paolo Cavallini
Il 24/10/2016 10:32, Giovanni Manghi ha scritto:

> attached). I have used it a few times in the past to change/create the
> descriptions files when a new SAGA version was published.

could you please do a PR instead of keeping them local?
Thanks.

-- 
Paolo Cavallini - www.faunalia.eu
QGIS & PostGIS courses: http://www.faunalia.eu/training.html
https://www.google.com/trends/explore?date=all=IT=qgis,arcgis
___
Qgis-developer mailing list
Qgis-developer@lists.osgeo.org
List info: http://lists.osgeo.org/mailman/listinfo/qgis-developer
Unsubscribe: http://lists.osgeo.org/mailman/listinfo/qgis-developer

Re: [Qgis-developer] Yet again: SAGA 2.3.1 fails

2016-10-24 Thread Giovanni Manghi
Hi,


> Unknown option 'INTERPOL'

The problem is already know, this parameter (in the io_gdal module,
the one to import raster layers) has changed name in recent SAGA
releases. This must be addressed also in SagaAlgorithm***.py

Regarding all other changes/additions that could have been made too
SAGA modules:
some time ago Victor (or AlexB I don't remember) provided a script
that that compares two different versions os SAGA (see example
attached). I have used it a few times in the past to change/create the
descriptions files when a new SAGA version was published.

-- G --


diff223_224
Description: Binary data
___
Qgis-developer mailing list
Qgis-developer@lists.osgeo.org
List info: http://lists.osgeo.org/mailman/listinfo/qgis-developer
Unsubscribe: http://lists.osgeo.org/mailman/listinfo/qgis-developer

Re: [Qgis-developer] Yet again: SAGA 2.3.1 fails

2016-10-22 Thread Paolo Cavallini
Il 22/10/2016 12:27, Sebastiaan Couwenberg ha scritto:
> On 10/22/2016 12:09 PM, Paolo Cavallini wrote:
>> Obviously removing
>>
>> -INTERPOL 0
>>
>> from
>>
>> /usr/share/qgis/python/plugins/processing/algs/saga/SagaAlgorithm214.py
>>
>> solves the issue, but a proper fix would involve supporting also 2.3.1
>> version. I know it has been a long and frustrating discussion. I'm
>> willing to add 2.31 support if noone objects.
> 
> Please do add support for SAGA 2.3.x LTS, it's the ideal target to
> support in QGIS.

One more issue: in FlowAccumulation(*).txt params have changed:

CAREA > FLOW (I think)
ACCU_TOT > ACCU_TOTAL

However, replacing the params returns:
Separator expected after the option 'FLOW'.

All the best.
-- 
Paolo Cavallini - www.faunalia.eu
QGIS & PostGIS courses: http://www.faunalia.eu/training.html
https://www.google.com/trends/explore?date=all=IT=qgis,arcgis
___
Qgis-developer mailing list
Qgis-developer@lists.osgeo.org
List info: http://lists.osgeo.org/mailman/listinfo/qgis-developer
Unsubscribe: http://lists.osgeo.org/mailman/listinfo/qgis-developer

Re: [Qgis-developer] Yet again: SAGA 2.3.1 fails

2016-10-22 Thread Sebastiaan Couwenberg
On 10/22/2016 12:09 PM, Paolo Cavallini wrote:
> Obviously removing
> 
> -INTERPOL 0
> 
> from
> 
> /usr/share/qgis/python/plugins/processing/algs/saga/SagaAlgorithm214.py
> 
> solves the issue, but a proper fix would involve supporting also 2.3.1
> version. I know it has been a long and frustrating discussion. I'm
> willing to add 2.31 support if noone objects.

Please do add support for SAGA 2.3.x LTS, it's the ideal target to
support in QGIS.

Kind Regards,

Bas

-- 
 GPG Key ID: 4096R/6750F10AE88D4AF1
Fingerprint: 8182 DE41 7056 408D 6146  50D1 6750 F10A E88D 4AF1
___
Qgis-developer mailing list
Qgis-developer@lists.osgeo.org
List info: http://lists.osgeo.org/mailman/listinfo/qgis-developer
Unsubscribe: http://lists.osgeo.org/mailman/listinfo/qgis-developer

Re: [Qgis-developer] Yet again: SAGA 2.3.1 fails

2016-10-22 Thread Paolo Cavallini
Il 22/10/2016 11:39, Paolo Cavallini ha scritto:
> Hi all,
> current version of SAGA fails on QGIS 2.16:

Obviously removing

-INTERPOL 0

from

/usr/share/qgis/python/plugins/processing/algs/saga/SagaAlgorithm214.py

solves the issue, but a proper fix would involve supporting also 2.3.1
version. I know it has been a long and frustrating discussion. I'm
willing to add 2.31 support if noone objects.
All the best.

-- 
Paolo Cavallini - www.faunalia.eu
QGIS & PostGIS courses: http://www.faunalia.eu/training.html
https://www.google.com/trends/explore?date=all=IT=qgis,arcgis
___
Qgis-developer mailing list
Qgis-developer@lists.osgeo.org
List info: http://lists.osgeo.org/mailman/listinfo/qgis-developer
Unsubscribe: http://lists.osgeo.org/mailman/listinfo/qgis-developer

[Qgis-developer] Yet again: SAGA 2.3.1 fails

2016-10-22 Thread Paolo Cavallini
Hi all,
current version of SAGA fails on QGIS 2.16:
===
io_gdal 0 -TRANSFORM 1 -INTERPOL 0 -GRIDS
"/tmp/processingb7e4b8e7fbdc4d579e1a8d72a9ec515a/5aa48aea26734ddcbb8ec9dba3b19ae2/dem25.sgrd"
-FILES
"/usr/local/src/qgis/QGIS-Training-Data/training_manual_data/processing/no_data/dem25.tif"
grid_calculus "Grid Calculator" -GRIDS
"/tmp/processingb7e4b8e7fbdc4d579e1a8d72a9ec515a/5aa48aea26734ddcbb8ec9dba3b19ae2/dem25.sgrd"
-FORMULA "a * 3.28084" -USE_NODATA false -TYPE 7 -RESULT
"/tmp/processingb7e4b8e7fbdc4d579e1a8d72a9ec515a/1289913a2637404886ea25cffd4845c5/RESULT.sdat"
Unknown option 'INTERPOL'


# ## # ##
### ### ## ###
### # ## ##  # ##
### # ## # #
# # ## # # ##


SAGA Version: 2.3.1
===
All the best.
-- 
Paolo Cavallini - www.faunalia.eu
QGIS & PostGIS courses: http://www.faunalia.eu/training.html
https://www.google.com/trends/explore?date=all=IT=qgis,arcgis
___
Qgis-developer mailing list
Qgis-developer@lists.osgeo.org
List info: http://lists.osgeo.org/mailman/listinfo/qgis-developer
Unsubscribe: http://lists.osgeo.org/mailman/listinfo/qgis-developer