#229 SAGA GIS v 2.2.6 and 2.2.7 valley depth module

[SteveP]

Hello,

It looks like an error has been introduced, maybe after v.2.2.3 with the valley depth module. The module fails to execute (either in the saga_gui or saga_cmd) using SAGA 2.2.6 and 2.2.7. This occurs irrespectively of grid size (i.e. it is not a memory issue), whereas the module appears to work fine in SAGA v. 2.2.3.

Also it looks like valley depth runs fine as part of the 'basic terrain analysis' suite, but not as a standalone module.

Steve

[Volker Wichmann]

Hi Steve,

thanks for reporting. I've just committed a fix: https://sourceforge.net/p/saga-gis/code-0/2897/

Best regards,
Volker

[Olaf Conrad]

hello steve, thanks volker for your fast fix. i was working on this bug at the same time and just committed my solution, because it also adds a fix for a correct ridge level calculation. in addition i created a tool chain that performs the same valley depth calculation (attached to this post), which you can download and use as temporary replacement for the original tool.
best regards,
olaf...

[SteveP]

Thank you Olaf and Volker for fixing this so quickly.

Yes I have been running valley depth manually, i.e. inverting the DEM, extracting channels and using the vertical distance to channel network tool, so thanks for the tool chain.

PS: do you have any plans to integrate Python directly into SAGA in the future, so that tools could be developed in Python and accessed directly within SAGA? Or perhaps you want to keep algorithms in C++.

Best regards,

Steve

Sent from my iPhone

On Apr 15, 2016, at 3:46 AM, Olaf Conrad oconrad@users.sf.net wrote:

hello steve, thanks volker for your fast fix. i was working on this bug at the same time and just committed my solution, because it also adds a fix for a correct ridge level calculation. in addition i created a tool chain that performs the same valley depth calculation (attached to this post), which you can download and use as temporary replacement for the original tool.
best regards,
olaf...

Attachments:

ta_valley_depth.xml (3.1 kB; text/xml)
[bugs:#229] SAGA GIS v 2.2.6 and 2.2.7 valley depth module

Status: closed
Group: v2.2.8
Created: Thu Apr 14, 2016 10:12 PM UTC by SteveP
Last Updated: Fri Apr 15, 2016 07:33 AM UTC
Owner: Olaf Conrad

Hello,

It looks like an error has been introduced, maybe after v.2.2.3 with the valley depth module. The module fails to execute (either in the saga_gui or saga_cmd) using SAGA 2.2.6 and 2.2.7. This occurs irrespectively of grid size (i.e. it is not a memory issue), whereas the module appears to work fine in SAGA v. 2.2.3.

Also it looks like valley depth runs fine as part of the 'basic terrain analysis' suite, but not as a standalone module.

Steve

Sent from sourceforge.net because you indicated interest in https://sourceforge.net/p/saga-gis/bugs/229/

To unsubscribe from further messages, please visit https://sourceforge.net/auth/subscriptions/

[Volker Wichmann]

Hi Steve,

there are no plans to integrate Python directly in SAGA. We only provide the possibilty to use the SAGA API in Python tools.

Best regards,
Volker