[GRASS-dev] Re: [GRASS GIS] #1059: wxGUI: No menu option for r.mapcalc

[GRASS GIS]

#1059: wxGUI: No menu option for r.mapcalc
-----------------------+----------------------------------------------------
  Reporter:  vesnikos  |       Owner:  martinl        
      Type:  defect    |      Status:  reopened       
  Priority:  major     |   Milestone:  7.0.0          
 Component:  wxGUI     |     Version:  svn-trunk      
Resolution:            |    Keywords:  r.mapcalc wxGUI
  Platform:  Linux     |         Cpu:  x86-32         
-----------------------+----------------------------------------------------
Comment (by martinl):

 Replying to [comment:16 mmetz]:
 > I was talking about r42182, before this flood of changes came in. My
 diff is against revision 42182, not revision 42210. In r42210, a lot of
 changes have been made to the GUI and the module, some of the changes have
 been changed again...

 Most of the changes are related to the major clean up of mcalc_builder.py
 which is unrelated to this ticket. It was just long-term TODO.

 > My point is that probably much less changes were required to make
 r.mapcalc visible in the wxGUI command search, that the GUI should be
 fixed first, before modules get broken by a quick

 Seems to work for me, http://gama.fsv.cvut.cz/~landa/grass/swf/wxgui-
 search-module.html.
 {{{r.mapcalc}}} has been added to the menu by changing menudata.xml,
 nothing more. Anyway sorry that I broke the {{{r.mapcalc}}} interface.

 > fix, that the two different interfaces for r.mapcalc are confusing, that
 more testing should be done on the local copy before a commit, and that
 the Map Calculator wxGUI interface is nice but unfinished work, no output,
 no help. I personally would recommend to go back to r42182 and start anew,
 looking for a solution that requires less modifications.

 I am not original author of wxGUI Map Calc. In any case it's a usable
 tool. Of course it needs various fixes, any volunteer welcomed... See
 simple example at http://gama.fsv.cvut.cz/~landa/grass/swf/wxgui-
 mapcalc.html. Anyway to have two items in the menu can be confusing.

 > Sorry for the ranting, but even though trunk is the development version,
 I would sometimes prefer some more testing before submitting changes, and
 a flood of commits for the same problem within one day clearly indicates
 that the first commits were not well thought through.

 wxGUI is quite experimental from my point of view. It's not part of
 'core', anyway this flood of commits refers mostly to clean up of
 mcalc_bulder.py module.

-- 
Ticket URL: <http://trac.osgeo.org/grass/ticket/1059#comment:17>
GRASS GIS <http://grass.osgeo.org>