[GRASS-dev] Re: [GRASS GIS] #1457: Zoom in Georectify Does not Work

[GRASS GIS]

#1457: Zoom in Georectify Does not Work
------------------------------+---------------------------------------------
 Reporter:  jjrambo           |       Owner:  grass-dev@…              
     Type:  defect            |      Status:  new                      
 Priority:  normal            |   Milestone:  7.0.0                    
Component:  wxGUI             |     Version:  svn-trunk                
 Keywords:  Zoom, Georectify  |    Platform:  Linux                    
      Cpu:  x86-32            |  
------------------------------+---------------------------------------------

Comment(by mmetz):

 Replying to [comment:12 annakrat]:
 > Replying to [comment:4 martinl]:
 > > Replying to [comment:2 mmetz]:
 > > > I guess the check for alignExtent must be disabled for the
 Georectifier. Martin, is this alignExtent thing something new or at least
 newer than the Georectifier?
 > >
 > > yes, see r47987. I think there is no need to disable this feature in
 georectifier(?) For quick `gcpmapdisp.py` code should be updated or
 rewrite code as I suggested and use common super-class for also for
 `mapdisp.MapFrame`. This avoid such bugs in the future and also made life
 easier for implementing similar tools, like wxGUI i.class, etc.
 >
 >
 > As you might have noticed we are working on this super-class. We found
 quite a lot of  code related to georectifier - GRMapWindow (especially in
 mapdisp.py, mapdisp_window.py), which seems to be useless. For example
 there are `toolbars['georect']` and `toolbars['gcpdisp']` (which is used
 now). Is this obsolete? If yes, we will clean the code.

 `toolbars['georect']` is unused, it is a left-over from the previous
 version of the georectifier. Only `toolbars['gcpdisp']` is used by the
 current georectifier.

 I will help adjusting the code as soon as I have a bit more breathing
 time.

 Markus M

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