[GRASS-dev] [GRASS GIS] #2901: Change r.watershed message type for flag a

GRASS GIS trac at osgeo.org
Mon Feb 8 14:00:43 PST 2016


#2901: Change r.watershed message type for flag a
-----------------------------------+---------------------------------
 Reporter:  isaacullah             |      Owner:  grass-dev@…
     Type:  defect                 |     Status:  new
 Priority:  normal                 |  Milestone:  7.0.4
Component:  Raster                 |    Version:  svn-releasebranch70
 Keywords:  r.watershed, messages  |        CPU:  x86-64
 Platform:  All                    |
-----------------------------------+---------------------------------
 Currently, using the flag -a in r.watershed ("Use positive flow
 accumulation even for likely underestimates") induces an output message at
 the GRASS_WARNING level:


 {{{
 WARNING: Writing out only positive flow accumulation values.
 WARNING: Cells with a likely underestimate for flow accumulation can no
 longer be identified.
 }}}


 That this message is at the WARNING level means that it will not be
 silenced in GRASS quiet mode. Can this warning message be changed from
 WARNING level to standard GRASS_MESSAGE level? The reason is that in user
 or addon scripts that want to do some iteration with r.watershed, this
 message will be repeatedly written out to the terminal. Not only is that
 annoying and washes out any more meaningful messages being output from the
 script, but we also loose some processing time to the many repeated screen
 writeouts of this message. If it was at the normal MESSAGE level, it would
 follow the typical GRASS verbosity settings, making it play along better
 in addon scripts.

 In any case, I think it shouldn't be at the WARNING level anyway, since it
 makes the less experienced GRASS user think an error has occurred, rather
 than acting as a simple reminder that you've checked a particular flag.

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/2901>
GRASS GIS <https://grass.osgeo.org>



More information about the grass-dev mailing list