#2848: g.gui.vdigit: snapping distance seems to only be defined in map units, not
screen pixels
--------------------------+---------------------------------
Reporter: mlennert | Owner: grass-dev@…
Type: defect | Status: new
Priority: major | Milestone: 7.0.5
Component: wxGUI | Version: svn-releasebranch70
Resolution: | Keywords: vector, digitize
CPU: Unspecified | Platform: Unspecified
--------------------------+---------------------------------
Comment (by mlennert):
Replying to [comment:10 annakrat]:
> Replying to [comment:9 mlennert]:
> > Replying to [comment:8 annakrat]:
> > > Replying to [comment:7 mlennert]:
> > > > Replying to [comment:6 annakrat]:
> > > > > Please test. It was only problem when using different locale.
> > > >
> > > > Right. It +/- works now, i.e. when I chose "screen pixels", I now
get the correct value in map units below. However, when I change the value
(i.e. the number of screen pixels), I get a wrong number below: the number
of screen pixels instead of map units. I have to switch to map units and
back to screen pixels for the depicted value in map units to be correct
again.
> > >
> > > OK, I committed a new fix in r69579.
> >
> > Seems to work great, now ! Thanks.
> >
> > Not sure if this is mature enough to backport to release70. You're a
better judge of that.
>
> The only problem is in the settings. The old value should be ignored and
then new value (0 or 1 is saved). You can test it by manually changing the
values in the settings file. Since you didn't report any problem with the
new patch I assume it ignores your saved settings value ("screen pixels"
or the french version), but please check, you should have now there 0 or 1
if you saved the settings.
I'm not sure I saved these settings before. Now, I do have
{{{
vdigit;snapping;units;0;value;20
}}}
However, when I now try to open the settings dialog in the grass 7.0.5
digitizer, I get:
{{{
Traceback (most recent call last):
File
"/data/home/mlennert/SRC/GRASS/grass70_release/dist.x86_64
-pc-linux-gnu/gui/wxpython/vdigit/toolbars.py", line 528, in
OnSettings
self.settingsDialog = VDigitSettingsDialog(parent =
self.parent, giface = self._giface)
File
"/data/home/mlennert/SRC/GRASS/grass70_release/dist.x86_64
-pc-linux-gnu/gui/wxpython/vdigit/preferences.py", line 43,
in __init__
self._createGeneralPage(notebook)
File
"/data/home/mlennert/SRC/GRASS/grass70_release/dist.x86_64
-pc-linux-gnu/gui/wxpython/vdigit/preferences.py", line 168,
in _createGeneralPage
self.snappingUnit.SetStringSelection(UserSettings.Get(group
= 'vdigit', key = "snapping", subkey = 'units'))
File "/usr/lib/python2.7/dist-
packages/wx-3.0-gtk2/wx/_core.py", line 13002, in
SetStringSelection
return _core_.ItemContainer_SetStringSelection(*args,
**kwargs)
TypeError
:
String or Unicode type required
}}}
so there is an issue of backwards compatibility....
--
Ticket URL: <https://trac.osgeo.org/grass/ticket/2848#comment:11>
GRASS GIS <https://grass.osgeo.org>