[GRASS-dev] [GRASS GIS] #839: wxGUI: About GRASS menu item broken

#839: wxGUI: About GRASS menu item broken
--------------------+-------------------------------------------------------
Reporter: hamish | Owner: grass-dev@lists.osgeo.org
     Type: defect | Status: new
Priority: major | Milestone: 6.5.0
Component: wxGUI | Version: svn-develbranch6
Keywords: | Platform: Linux
      Cpu: x86-64 |
--------------------+-------------------------------------------------------
Hi

in the 6.5 wxGUI if I do `Help -> About GRASS GIS` I get this error in the
layer manager output tab:

{{{
Traceback (most recent call last):
   File "/usr/src/grass/svn/grass65/dist.x86_64
-unknown-linux-gnu/etc/wxpython/wxgui.py", line 554, in
OnAboutGRASS

win = AboutWindow(self)
   File "/usr/src/grass/svn/grass65/dist.x86_64
-unknown-linux-gnu/etc/wxpython/gui_modules/help.py", line
406, in __init__

transwin = self.PageTranslators()
   File "/usr/src/grass/svn/grass65/dist.x86_64
-unknown-linux-gnu/etc/wxpython/gui_modules/help.py", line
601, in PageTranslators

label = name))
   File "/usr/lib/python2.5/site-
packages/wx-2.8-gtk2-unicode/wx/_controls.py", line 1136, in
__init__

_controls_.StaticText_swiginit(self,_controls_.new_StaticTex
t(*args, **kwargs))
UnicodeDecodeError
:
'ascii' codec can't decode byte 0xc3 in position 1: ordinal
not in range(128)
}}}

?,
Hamish

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/839&gt;
GRASS GIS <http://grass.osgeo.org>

#839: wxGUI: About GRASS menu item broken
---------------------+------------------------------------------------------
  Reporter: hamish | Owner: grass-dev@lists.osgeo.org
      Type: defect | Status: new
  Priority: major | Milestone: 6.5.0
Component: wxGUI | Version: svn-develbranch6
Resolution: | Keywords:
  Platform: Linux | Cpu: x86-64
---------------------+------------------------------------------------------
Comment (by cmbarton):

It looks like this could be fixed in the most up-to-date svn version of
6.5. When I look at line 601 it is different that the line reported in
this error.

Michael

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/839#comment:1&gt;
GRASS GIS <http://grass.osgeo.org>

#839: wxGUI: About GRASS menu item broken
---------------------+------------------------------------------------------
  Reporter: hamish | Owner: grass-dev@lists.osgeo.org
      Type: defect | Status: new
  Priority: major | Milestone: 6.5.0
Component: wxGUI | Version: svn-develbranch6
Resolution: | Keywords:
  Platform: Linux | Cpu: x86-64
---------------------+------------------------------------------------------
Comment (by hamish):

nope, still broken today. Now it says line 550, 409, 604, 1136
respectively.

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/839#comment:2&gt;
GRASS GIS <http://grass.osgeo.org>

#839: wxGUI: About GRASS menu item broken
---------------------+------------------------------------------------------
  Reporter: hamish | Owner: grass-dev@lists.osgeo.org
      Type: defect | Status: new
  Priority: major | Milestone: 6.5.0
Component: wxGUI | Version: svn-develbranch6
Resolution: | Keywords:
  Platform: Linux | Cpu: x86-64
---------------------+------------------------------------------------------
Comment (by cmbarton):

I found the location but have no idea why you are getting an error.

I also found out that on the Mac these new files translator.cvs and
contributor.cvs are not getting moved to $GISBASE/etc to be found by About
GRASS. I moved them manually to test and still got no error. There is
nothing odd about the code.

Has anyone else had an error with About GRASS?

Michael

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/839#comment:3&gt;
GRASS GIS <http://grass.osgeo.org>

#839: wxGUI: About GRASS menu item broken
---------------------+------------------------------------------------------
  Reporter: hamish | Owner: grass-dev@lists.osgeo.org
      Type: defect | Status: new
  Priority: major | Milestone: 6.5.0
Component: wxGUI | Version: svn-develbranch6
Resolution: | Keywords:
  Platform: Linux | Cpu: x86-64
---------------------+------------------------------------------------------
Comment (by marisn):

Replying to [comment:2 hamish]:
> nope, still broken today. Now it says line 550, 409, 604, 1136
respectively.
Could this be locale-dependent issue? Michael is using en_US locale and
thus all unicode/anycode text == ascii, which may not be true for other
locales. Just guessing without reading code.

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/839#comment:4&gt;
GRASS GIS <http://grass.osgeo.org>

#839: wxGUI: About GRASS menu item broken
---------------------+------------------------------------------------------
  Reporter: hamish | Owner: grass-dev@lists.osgeo.org
      Type: defect | Status: new
  Priority: major | Milestone: 6.5.0
Component: wxGUI | Version: svn-develbranch6
Resolution: | Keywords:
  Platform: Linux | Cpu: x86-64
---------------------+------------------------------------------------------
Comment (by marisn):

Ah. Sorry. Pasted reply without testing.
Michael:
{{{
export LC_ALL=de_DE
export LANG=de_DE
g.gui gui=wxpython
}}}

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/839#comment:5&gt;
GRASS GIS <http://grass.osgeo.org>

#839: wxGUI: About GRASS menu item broken
---------------------+------------------------------------------------------
  Reporter: hamish | Owner: grass-dev@lists.osgeo.org
      Type: defect | Status: new
  Priority: major | Milestone: 6.5.0
Component: wxGUI | Version: svn-develbranch6
Resolution: | Keywords:
  Platform: Linux | Cpu: x86-64
---------------------+------------------------------------------------------
Comment (by hamish):

Replying to [comment:4 marisn]:
> Could this be locale-dependent issue? Michael is using en_US
> locale and thus all unicode/anycode text == ascii, which may not
> be true for other locales. Just guessing without reading code.

perhaps, but I don't think so. I'm pretty sure the workstation I was
testing on before is set up as LANG=C, and on a second machine here I'm a
little surprised to see it is actually set to en_NZ instead of C or en_US.
(usually it's just less trouble that way, and dealing with the spelling
differences isn't a big deal, I just resolve myself to spelling poorly in
both contexts :wink:

On this second machine it works -- same debian version, only difference
AFAIK is 32bit P4 instead of 64bit AMD. shrug. does it fail for anyone
else? ubuntu users?

I also found that the 2 .csv files were missing, but the translators and
contributers tabs are missing unless I drag the window wider. also the
translators tab is open upon launch, but the info one should be. also I
find scrolling with the mouse wheel down the gpl and authors list that the
text is ''highly'' jumpy. is "contributers" redundant; shouldn't we just
take care to merge that into Authors? (?)

Hamish

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/839#comment:6&gt;
GRASS GIS <http://grass.osgeo.org>

#839: wxGUI: About GRASS menu item broken
---------------------+------------------------------------------------------
  Reporter: hamish | Owner: grass-dev@lists.osgeo.org
      Type: defect | Status: new
  Priority: major | Milestone: 6.5.0
Component: wxGUI | Version: svn-develbranch6
Resolution: | Keywords:
  Platform: Linux | Cpu: x86-64
---------------------+------------------------------------------------------
Comment (by hamish):

{{{
_controls_.StaticText_swiginit(self,_controls_.new_StaticTex
t(*args, **kwargs))
UnicodeDecodeError
:
'ascii' codec can't decode byte 0xc3 in position 1: ordinal
not in range(128)
}}}

Ok maybe I can expand on Maris's theory. What if the translators.csv file
has unicode symbols in it, but my LANG=C monolanguage install is too
ignorant to deal with that?

I see non-ascii chars in the names of the Vietnamese and Czech
translators. I bet if I anglify those it'll work.

Hamish

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/839#comment:7&gt;
GRASS GIS <http://grass.osgeo.org>

#839: wxGUI: About GRASS menu item broken
---------------------+------------------------------------------------------
  Reporter: hamish | Owner: martinl
      Type: defect | Status: assigned
  Priority: major | Milestone: 6.5.0
Component: wxGUI | Version: svn-develbranch6
Resolution: | Keywords:
  Platform: Linux | Cpu: x86-64
---------------------+------------------------------------------------------
Changes (by martinl):

* cc: grass-dev@lists.osgeo.org (added)
  * owner: grass-dev@lists.osgeo.org => martinl
  * status: new => assigned

Comment:

Replying to [comment:7 hamish]:
{{{
> _controls_.StaticText_swiginit(self,_controls_.new_StaticTex
> t(*args, **kwargs))
> UnicodeDecodeError
> :
> 'ascii' codec can't decode byte 0xc3 in position 1: ordinal
> not in range(128)
}}}
>
> Ok maybe I can expand on Maris's theory. What if the translators.csv
file has unicode symbols in it, but my LANG=C monolanguage install is too
ignorant to deal with that?
>
> I see non-ascii chars in the names of the Vietnamese and Czech
translators. I bet if I anglify those it'll work.

Right, hopefully fixed in r40034.

Martin

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/839#comment:8&gt;
GRASS GIS <http://grass.osgeo.org>

#839: wxGUI: About GRASS menu item broken
---------------------+------------------------------------------------------
  Reporter: hamish | Owner: martinl
      Type: defect | Status: assigned
  Priority: major | Milestone: 6.5.0
Component: wxGUI | Version: svn-develbranch6
Resolution: | Keywords:
  Platform: Linux | Cpu: x86-64
---------------------+------------------------------------------------------
Comment (by hamish):

Martin:
> Right, hopefully fixed in r40034.

nice, it now launches.

still todo:
  - the stuttering effect is still quite bad for all scrolling pages,
leaving garbage in its wake for the new .csv tabs only. see attached
screenshot
  - you'll notice in that screenshot that the Translators tab is missing.
if the widget used to make the tabs doesn't have <- arrows -> to move onto
extra tabs, the only thing I can think of is to make the default window
size a bit wider. On this one machine it has to be 600px wide for all tabs
to display. I think on the other machine I tried it on the number was
about 640px. I guess it depends on local font size.
  - the version screen should come up first when you open the window, not
the translators list.

thanks,
Hamish

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/839#comment:9&gt;
GRASS GIS <http://grass.osgeo.org>

#839: wxGUI: About GRASS menu item broken
---------------------+------------------------------------------------------
  Reporter: hamish | Owner: martinl
      Type: defect | Status: assigned
  Priority: major | Milestone: 6.5.0
Component: wxGUI | Version: svn-develbranch6
Resolution: | Keywords:
  Platform: Linux | Cpu: x86-64
---------------------+------------------------------------------------------
Comment (by martinl):

Replying to [comment:9 hamish]:
> still todo:
> - the stuttering effect is still quite bad for all scrolling pages,
leaving garbage in its wake for the new .csv tabs only. see attached
screenshot

It's seems to be a wxPython bug.

> - you'll notice in that screenshot that the Translators tab is missing.
if the widget used to make the tabs doesn't have <- arrows -> to move onto
extra tabs, the only thing I can think of is to make the default window
size a bit wider. On this one machine it has to be 600px wide for all tabs
to display. I think on the other machine I tried it on the number was
about 640px. I guess it depends on local font size.

done in r40056.

> - the version screen should come up first when you open the window, not
the translators list.

should be done automatically, in r40057 is called {{{SetSelection}}}
explicitly.

Martin

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/839#comment:10&gt;
GRASS GIS <http://grass.osgeo.org>

#839: wxGUI: About GRASS menu item broken
---------------------+------------------------------------------------------
  Reporter: hamish | Owner: martinl
      Type: defect | Status: assigned
  Priority: major | Milestone: 6.5.0
Component: wxGUI | Version: svn-develbranch6
Resolution: | Keywords:
  Platform: Linux | Cpu: x86-64
---------------------+------------------------------------------------------
Comment (by hamish):

> Replying to [comment:9 hamish]:
> > - the stuttering effect

Replying to [comment:10 martinl]:
> It's seems to be a wxPython bug.

ok, will ignore.

> > - you'll notice in that screenshot that the Translators tab
> > is missing.
> done in r40056.

now it's obvious; thanks.

> > - the version screen should come up first when you open
> > the window, not the translators list.
>
> should be done automatically, in r40057 is called
> {{{SetSelection}}} explicitly.

now works for me, thanks.

Hamish

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/839#comment:11&gt;
GRASS GIS <http://grass.osgeo.org>

#839: wxGUI: About GRASS menu item broken
---------------------+------------------------------------------------------
  Reporter: hamish | Owner: martinl
      Type: defect | Status: closed
  Priority: major | Milestone: 6.5.0
Component: wxGUI | Version: svn-develbranch6
Resolution: fixed | Keywords:
  Platform: Linux | Cpu: x86-64
---------------------+------------------------------------------------------
Changes (by martinl):

  * status: assigned => closed
  * resolution: => fixed

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/839#comment:12&gt;
GRASS GIS <http://grass.osgeo.org>