Syntax check for raster calculator - Githubissues

qgis / QGIS

QGIS is a free, open source, cross platform (lin/win/mac) geographical information system (GIS)

https://qgis.org

GNU General Public License v2.0

10.54k stars 2.99k forks source link

Syntax check for raster calculator #19753

Open qgib opened 10 years ago

qgib commented 10 years ago

Author Name: Paolo Cavallini (@pcav) Original Redmine Issue: 11483

Redmine category:processing/core

It would be good to add a syntax checker for raster calculator, so the user is prevented before running a wrong formula. Please see the QGIS raster calc or the expression builder for examples. Aslo, a syntax highlight would be good to have.

qgib commented 7 years ago

Author Name: Giovanni Manghi (@gioman)

easy_fix was configured as 0

qgib commented 6 years ago

Author Name: Paolo Cavallini (@pcav)

Still true in QGIS 3.

assigned_to_id removed Victor Olaya
category_id was changed from Processing/SAGA to Processing/Core

ghtmtt commented 4 years ago

@gioman I think this is fixed now, right?

gioman commented 4 years ago

@ghtmtt using the one in Processing a user is not prevented to run the raster calc even if the GUI shows that the formula is wrong. On the other hand using the one in the "raster menu" if the formula is wrong the "run" button is disabled.

Regardless both show a message (when the formula is wrong) so at least is partially fixed/implemented. I still wonder why having two implementations of the same tool, while the tendency has been to move all this type of tools to Processing only.

AlisterH commented 6 months ago

@ghtmtt using the one in Processing a user is not prevented to run the raster calc even if the GUI shows that the formula is wrong.

For the record, when running from processing toolbox I don't see any formula check - I presume that comment was prior to the rewrite of the raster calculator, and this is a "regression".

But either this or #55386 should probably be closed as a duplicate.