<div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div>Hi Nyall,</div><div><br></div><div>I think this rises another problem, that is the use of a very old version of SAGA (2.3.2) in QGIS.</div><div><br></div><div>I've just tested Difference algorithm with Lene dataset in native SAGA 2.3.2 and 7.2.0, and 2.3.2 gives the problem described, but 7.2.0 gives the correct result:</div><div><br></div><div><a href="https://cld.pt/dl/download/2dfb0db5-eb74-4396-ad18-7451fd1fe788/test_saga_232.jpg">https://cld.pt/dl/download/2dfb0db5-eb74-4396-ad18-7451fd1fe788/test_saga_232.jpg</a></div><div><br></div><div><a href="https://cld.pt/dl/download/92182907-d37a-4441-891c-3c85064b3100/test_saga_720.jpg">https://cld.pt/dl/download/92182907-d37a-4441-891c-3c85064b3100/test_saga_720.jpg</a> <br></div><div><br></div><div>So, we are faced again with the question of keep an old version of SAGA in QGIS core, or make it as an external plugin, with someone keeping an eye in the changes introduced by SAGA team between versions (as we saw in the past). <br></div><div><br></div><div>Alexander Bruy had made an effort to keep a parallel SAGA plugin that support newer SAGA versions: <a href="https://plugins.bruy.me/processing-saga.html">https://plugins.bruy.me/processing-saga.html</a> <br></div><div>But it is not in the official repository. Maybe this is the time to discuss again the use of SAGA as an external plugin?<br></div><div><br></div><div>Best regards,</div><div>Pedro Venâncio</div><div><br></div><div><br></div><div><br></div></div></div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Nyall Dawson <<a href="mailto:nyall.dawson@gmail.com">nyall.dawson@gmail.com</a>> escreveu no dia segunda, 4/03/2019 à(s) 11:11:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi list,<br>
<br>
Just raising the discussion about what we should do with SAGA<br>
Processing algorithms when the results generated by SAGA are known to<br>
be incorrect (i.e. there's a bug in SAGA itself).<br>
<br>
Specifically, the SAGA difference and symmetric difference algorithms<br>
generate incorrect results. See <a href="https://issues.qgis.org/issues/21354" rel="noreferrer" target="_blank">https://issues.qgis.org/issues/21354</a>,<br>
and the results from<br>
<a href="https://github.com/qgis/QGIS/blob/master/python/plugins/processing/tests/testdata/saga_algorithm_tests.yaml#L318" rel="noreferrer" target="_blank">https://github.com/qgis/QGIS/blob/master/python/plugins/processing/tests/testdata/saga_algorithm_tests.yaml#L318</a><br>
. This seems to be a general problem with the algorithms, not isolated<br>
to particular input data.<br>
<br>
My gut feeling is: we should "deprecate" these algorithms (which means<br>
they continue to work for existing models, but are hidden from the UI<br>
and cannot be run from the toolbox or added to new models). I don't<br>
see the value in exposing broken algorithms when we have robust<br>
alternatives (the native QGIS difference/symmetric difference<br>
algorithms are very fast, stable, and well tested), and potentially a<br>
lot of harm.<br>
<br>
Thoughts?<br>
<br>
Nyall<br>
_______________________________________________<br>
QGIS-Developer mailing list<br>
<a href="mailto:QGIS-Developer@lists.osgeo.org" target="_blank">QGIS-Developer@lists.osgeo.org</a><br>
List info: <a href="https://lists.osgeo.org/mailman/listinfo/qgis-developer" rel="noreferrer" target="_blank">https://lists.osgeo.org/mailman/listinfo/qgis-developer</a><br>
Unsubscribe: <a href="https://lists.osgeo.org/mailman/listinfo/qgis-developer" rel="noreferrer" target="_blank">https://lists.osgeo.org/mailman/listinfo/qgis-developer</a></blockquote></div>