[GRASS-dev] [bug #5471] (grass) d.vect too noisy

GRASS GIS GRASS GIS developer
1

this bug's URL: http://intevation.de/rt/webrt?serial_num=5471
-------------------------------------------------------------------------

Subject: d.vect too noisy

G63> d.erase
G63> d.rast elevation.dem
G63> d.vect bugsites icon=basic/diamond col=white fcol=red
WARNING: Line color and background color are the same!

or

G63> d.erase blue
G63> d.vect bugsites icon=basic/diamond col=white fcol=red
WARNING: Line color and background color are the same!

If that warning needs to exist at all, it should check to see if the monitor
command list is empty or there are no raster maps displayed (eg d.what.rast)
before doing so.

Otherwise it is unneeded noise in the terminal whenever you use the color
white for something -- which may make a more important error message harder to
spot.

thanks,
Hamish

-------------------------------------------- Managed by Request Tracker

2

Request Tracker wrote:

this bug's URL: http://intevation.de/rt/webrt?serial_num=5471
-------------------------------------------------------------------------

Subject: d.vect too noisy

G63> d.erase
G63> d.rast elevation.dem
G63> d.vect bugsites icon=basic/diamond col=white fcol=red
WARNING: Line color and background color are the same!

or

G63> d.erase blue
G63> d.vect bugsites icon=basic/diamond col=white fcol=red
WARNING: Line color and background color are the same!

If that warning needs to exist at all, it should check to see if the monitor
command list is empty or there are no raster maps displayed (eg d.what.rast)
before doing so.

That still won't catch the case where you're using the PNG driver with
GRASS_BACKGROUNDCOLOR or GRASS_TRANSPARENT. There are probably other
similar cases.

IMHO, the warning should simply be removed.

--
Glynn Clements <glynn@gclements.plus.com>

3

Hi,

I think it could be removed. If not I can imagine

WARNING: Line color and *default* background color are the same

only in *MAXLVEL* of verbosity. But I am not sure if this warning
makes sense to the user.

Martin

2007/2/5, Glynn Clements <glynn@gclements.plus.com>:

Request Tracker wrote:

> this bug's URL: http://intevation.de/rt/webrt?serial_num=5471
> -------------------------------------------------------------------------
>
> Subject: d.vect too noisy
>
> G63> d.erase
> G63> d.rast elevation.dem
> G63> d.vect bugsites icon=basic/diamond col=white fcol=red
> WARNING: Line color and background color are the same!
>
> or
>
> G63> d.erase blue
> G63> d.vect bugsites icon=basic/diamond col=white fcol=red
> WARNING: Line color and background color are the same!
>
> If that warning needs to exist at all, it should check to see if the monitor
> command list is empty or there are no raster maps displayed (eg d.what.rast)
> before doing so.

That still won't catch the case where you're using the PNG driver with
GRASS_BACKGROUNDCOLOR or GRASS_TRANSPARENT. There are probably other
similar cases.

IMHO, the warning should simply be removed.

--
Glynn Clements <glynn@gclements.plus.com>

_______________________________________________
grass-dev mailing list
grass-dev@grass.itc.it
http://grass.itc.it/mailman/listinfo/grass-dev

--
Martin Landa <landa.martin@gmail.com> * http://gama.fsv.cvut.cz/~landa *