Opened 6 years ago

Closed 6 years ago

#2289 closed defect (fixed)

v.colors not working

Reported by: veroandreo Owned by: grass-dev@…
Priority: normal Milestone: 7.0.0
Component: Default Version: svn-trunk
Keywords: v.colors Cc: mlennert@…
CPU: x86-64 Platform: Linux

Description

Hi all,

I guess something is wrong with v.colors. I'm not able to colorize anything, it does say it sets the corresponding color table, but then... just grey...

v.colors map=soils_general@PERMANENT use=cat color=wave
d.mon wx0
d.vect soils_general

---> no colors, just grey polygons

v.colors map=firestations use=cat color=random
d.mon wx1 
d.vect firestations icon=basic/circle size=12 

---> all grey circles

It doesn't work from the gui either.

I'm using grass7.1.svn (r60164M) under fedora 20.

Best,

Vero

Change History (9)

comment:1 Changed 6 years ago by hcho

Resolution: fixed
Status: newclosed

Fixed in r60225.

comment:2 in reply to:  1 Changed 6 years ago by neteler

Milestone: 7.1.07.0.0
Version: unspecifiedsvn-trunk

Replying to hcho:

Fixed in r60225.

Please let's keep it open unless the backport is done

comment:3 Changed 6 years ago by neteler

Resolution: fixed
Status: closedreopened

comment:4 in reply to:  1 ; Changed 6 years ago by mlennert

Replying to hcho:

Fixed in r60225.

I can confirm that this fixes the issue, but how come that in grass70release the same code (as before the change) seems to work, but not in trunk ?

Moritz

comment:5 in reply to:  4 Changed 6 years ago by neteler

Replying to mlennert:

Replying to hcho:

Fixed in r60225.

I can confirm that this fixes the issue, but how come that in grass70release the same code (as before the change) seems to work, but not in trunk ?

Indeed - is a backport needed? Seems to work without in relbranch7.

comment:6 Changed 6 years ago by hcho

You should backport both r60149 and r60225, or none of them. At this point, if we don't already have r60149 in relbranch7, I wouldn't backport r60225 until we come up with a better solution.

comment:7 in reply to:  6 Changed 6 years ago by neteler

Replying to hcho:

You should backport both r60149 and r60225, or none of them. At this point, if we don't already have r60149 in relbranch7,

By chance I backported that earlier today in r61437.

I wouldn't backport r60225 until we come up with a better solution.

I'll leave that decision to you.

comment:8 Changed 6 years ago by hcho

Backported in r61449.

comment:9 Changed 6 years ago by neteler

Resolution: fixed
Status: reopenedclosed

Works fine now, closing.

Note: See TracTickets for help on using tickets.