Opened 14 years ago

Closed 13 years ago

#1014 closed defect (fixed)

wxgui: display labels doesn't work

Reported by: hamish Owned by: martinl
Priority: major Milestone: 6.4.2
Component: wxGUI Version: svn-develbranch6
Keywords: d.labels Cc: grass-dev@…
CPU: x86-32 Platform: Linux

Description

as reported on grass-user:

> I find the label layer I generated with v.label. 
> Here I get this error message:
> 
> --%<--
> Error
> Map <> not found.
> --%<--

spearfish example:

add grid or labels overlay button -> add labels -> name of label file -> rush.lbl -> [ok]

then you get the error shown above.

see OnAddLabels() in wxgui.py and gui_modules/wxgui_utils.py

Hamish

Attachments (1)

wxgui_dlabels.patch (578 bytes ) - added by ttalkington 14 years ago.
utils.py patch

Download all attachments as: .zip

Change History (7)

comment:1 by epatton, 14 years ago

Slightly related, the order of v.label and v.label.sa (what does the 'sa' stand for?) appear to be mixed up. Selecting 'Create optimally placed labels' launches v.label, and vice-versa.

~ Eric.

in reply to:  1 comment:2 by hamish, 14 years ago

Replying to epatton:

Slightly related, the order of v.label and v.label.sa ... appear to be mixed up. Selecting 'Create optimally placed labels' launches v.label, and vice-versa.

still in the latest SVN? I though we fixed that some months ago.

(what does the 'sa' stand for?)

"Simulated Annealing" http://en.wikipedia.org/wiki/Simulated_Annealing

Hamish

by ttalkington, 14 years ago

Attachment: wxgui_dlabels.patch added

utils.py patch

comment:3 by ttalkington, 14 years ago

The attached patch (wxgui_dlabels.patch) fixes this issue for me in the latest 7.0 svn. I'm not very familiar with the specifics of the wxGUI so this may be just a work around rather then a proper fix.

Taylor

comment:4 by mlennert, 13 years ago

The patch proposed in this ticket works well for me. Any reasons not to apply it ?

Moritz

in reply to:  4 comment:5 by martinl, 13 years ago

Cc: grass-dev@… added
Milestone: 6.4.06.4.2
Owner: changed from grass-dev@… to martinl
Status: newassigned

Replying to mlennert:

The patch proposed in this ticket works well for me. Any reasons not to apply it ?

Applied in r48431 (backported to devbr6 and relbr64).

comment:6 by martinl, 13 years ago

Resolution: fixed
Status: assignedclosed
Note: See TracTickets for help on using tickets.