Opened 6 years ago

Closed 5 years ago

Last modified 5 years ago

#3151 closed defect (fixed)

raster digitizer crashes GUI in trunk

Reported by: cmbarton Owned by: grass-dev@…
Priority: normal Milestone: 7.4.1
Component: wxGUI Version: unspecified
Keywords: raster digitizer Cc:
CPU: Unspecified Platform: MacOSX


When the raster digitizer is selected, it generates the error below and then locks ups the GUI so that you can't even quit it without killing Python from the OS.


Traceback (most recent call last):
  File "/Users/cmbarton/Dropbox (ASU)/GRASS_dropbox/GRASS-7.", line
255, in OnSelectTool

  File "/Users/cmbarton/Dropbox (ASU)/GRASS_dropbox/GRASS-7.", line
1507, in AddRDigit

  File "/Users/cmbarton/Dropbox (ASU)/GRASS_dropbox/GRASS-7.", line
56, in __init__

self._previousMap = self._mapSelectionCombo.GetValue()
  File "/Users/cmbarton/Dropbox (ASU)/GRASS_dropbox/GRASS-7.", line 610,
in GetValue

return _controls_.ComboBox_GetValue(*args, **kwargs)
C++ assertion "IsValid(n)" failed at /BUILD/wxPython-
src- in GetString():
wxChoice::GetString(): invalid index

I don't know if this is just a Mac issue or more general.

Change History (10)

comment:1 by cmbarton, 6 years ago

Forgot to add: compiled from trunk a few hours ago, version: 69469

comment:2 by annakrat, 6 years ago

Keywords: raster digitizer added
Platform: UnspecifiedMacOSX

comment:3 by neteler, 5 years ago

Can MacOSX please try the raster digitizer again?

comment:4 by annakrat, 5 years ago

Hmm, I haven't found time to fix this yet, so testing won't help I am afraid...

comment:5 by neteler, 5 years ago


Ticket retargeted after milestone closed

comment:6 by annakrat, 5 years ago

I didn't get this error anymore, but I had other problems with the digitizer toolbar, which should be fixed in r72793.

comment:7 by cmbarton, 5 years ago

Resolution: fixed
Status: newclosed

Just recompiled. This is fixed. Thanks much.

comment:8 by annakrat, 5 years ago

In 72854:

wxGUI/rdigit: fix raster digitizer toolbar on mac (merge from trunk, see #3151)

comment:9 by annakrat, 5 years ago

In 72855:

wxGUI: replace deprecated checkbox event method (merge from trunk, see #3151)

comment:10 by cmbarton, 5 years ago

Thanks. I am traveling the next couple weeks but will test these as soon as I can.

Note: See TracTickets for help on using tickets.