[GRASS-dev] Re: [GRASS GIS] #842: wx location wizard: spincontrol busted for UTM zone

[GRASS GIS]

#842: wx location wizard: spincontrol busted for UTM zone
-----------------------+----------------------------------------------------
  Reporter:  hamish    |       Owner:  grass-dev at lists.osgeo.org
      Type:  defect    |      Status:  new                      
  Priority:  critical  |   Milestone:  6.4.0                    
 Component:  wxGUI     |     Version:  svn-develbranch6         
Resolution:            |    Keywords:  location wizard, utm     
  Platform:  Linux     |         Cpu:  x86-64                   
-----------------------+----------------------------------------------------
Comment (by cmbarton):

 Replying to [comment:18 hamish]:
 > Replying to [comment:17 cmbarton]:
 > > I've changed the UTM Zone entry widget to a simple text widget
 > > with validation to keep it within the 1-60 range. I've added a
 > > new patch here: location_wizard.patch3. Please test.
 >
 > Yes it works if you type in zone=1-60.
 >
 > I'm seeing that +lon_0=-183 again though and I figured out how to
 reproduce it. If you enter "0" or "99" for the zone it automatically
 resets to zone 1 or 60, respectively. Apparently this is only visually. If
 you proceed to the summary page in the location wizard instead of +zone=23
 you will see +proj=tmerc +lon_0=411 etc. ...
 >
 > {{{
 > -PROJ_INFO-------------------------------------------------
 > name       : Transverse Mercator
 > proj       : tmerc
 > datum      : wgs84
 > ellps      : wgs84
 > lat_0      : 0
 > lon_0      : -183
 > k          : 0.9996
 > x_0        : 500000
 > y_0        : 0
 > no_defs    : defined
 > -PROJ_UNITS------------------------------------------------
 > unit       : Meter
 > units      : Meters
 > meters     : 1
 > }}}
 >
 >

 This is all fixed and now committed. I'll try to port this to to 6.5 and 7
 after it's had a few more tests--and will then mark this resolved. Thanks
 for all your help in getting this worked out. The bindings and events
 generated by the spinctrl were byzantine. This works much better.


 > fwiw1: I notice that PROJ_UNITS is uppercased, which seems different to
 how I always seem to have remembered it there.

 I don't understand what you're referring to. There is no upper() function
 in the module and nothing named proj_units that I could find. The location
 wizard does not set the proj units. This is done by g.proj when the proj4
 parameters are sent to it. I use the lower() function to make searching
 more rewarding regardless of case.


 >
 > fwiw2: I notice this patch no longer has the try: row/column stuff.
 > (different things should go into 2 different commits I guess)

 I had just archived this aside until I got the spin box fixed. It is in
 the bugfix commit.

 Michael

 >
 >
 > Hamish

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