Opened 12 years ago

Closed 10 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 11 years ago.
utils.py patch

Download all attachments as: .zip

Change History (7)

comment:1 Changed 11 years ago by epatton

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.

comment:2 in reply to:  1 Changed 11 years ago by hamish

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

Changed 11 years ago by ttalkington

Attachment: wxgui_dlabels.patch added

utils.py patch

comment:3 Changed 11 years ago by ttalkington

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 Changed 10 years ago by mlennert

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

Moritz

comment:5 in reply to:  4 Changed 10 years ago by martinl

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 Changed 10 years ago by martinl

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