Opened 12 years ago

Closed 11 years ago

#1093 closed defect (fixed)

wxModeler does not accept r.mapcalc action

Reported by: timmie Owned by: martinl
Priority: normal Milestone: 6.5.0
Component: wxGUI Version: svn-develbranch6
Keywords: modeler Cc: grass-dev@…
CPU: Unspecified Platform: Linux

Description

Trying the modeler, it comes not possible to add a action using the r.mapcalc module.

Change History (5)

comment:1 by martinl, 12 years ago

Cc: grass-dev@… added
Keywords: modeler added

The reason is that r.mapcalc in GRASS 6.5 doesn't use the parser, in the result wxGUI cannot generated dialog for this command. Try r.mapcalculator instead. This problem is not related to GRASS 7 (r.mapcalc uses in GRASS 7 the parser).

comment:2 by martinl, 12 years ago

Owner: changed from grass-dev@… to martinl
Status: newassigned

comment:3 by martinl, 12 years ago

Version: unspecifiedsvn-develbranch6

in reply to:  1 ; comment:4 by glynn, 12 years ago

Replying to martinl:

The reason is that r.mapcalc in GRASS 6.5 doesn't use the parser, in the result wxGUI cannot generated dialog for this command. Try r.mapcalculator instead. This problem is not related to GRASS 7 (r.mapcalc uses in GRASS 7 the parser).

If you want "raw" r.mapcalc from the GUI, it wouldn't be particularly hard to add a front-end script which implements the 7.0 semantics (i.e. expression= and file= options). E.g. (untested, boilerplate omitted):

if [ -n "$GIS_OPT_EXPRESSION" ] ; then
    exec r.mapcalc "$GIS_OPT_EXPRESSION"
elif [ -n "$GIS_OPT_FILE" ] ; then
    exec r.mapcalc < "$GIS_OPT_FILE"
else
    echo "Either expression= or file= must be given" >&2
    exit 1
fi

in reply to:  4 comment:5 by martinl, 11 years ago

Resolution: fixed
Status: assignedclosed

Replying to glynn:

If you want "raw" r.mapcalc from the GUI, it wouldn't be particularly hard to add a front-end script which implements the 7.0 semantics (i.e. expression= and file= options). E.g. (untested, boilerplate omitted):

done in r48861.

Note: See TracTickets for help on using tickets.