[GRASS-dev] [GRASS GIS] #3100: v.vol.rst WARNING in GRASS 7 still says cellout and cellinp rasters

GRASS GIS trac at osgeo.org
Tue Nov 22 03:51:58 PST 2016


#3100: v.vol.rst WARNING in GRASS 7 still says cellout and cellinp rasters
------------------------+-------------------------
  Reporter:  rorschach  |      Owner:  grass-dev@…
      Type:  defect     |     Status:  new
  Priority:  normal     |  Milestone:  7.0.6
 Component:  Default    |    Version:  7.0.4
Resolution:             |   Keywords:
       CPU:  x86-64     |   Platform:  Linux
------------------------+-------------------------

Comment (by mlennert):

 Replying to [comment:6 mlennert]:
 > Replying to [comment:4 neteler]:
 > > Attempt done in r69439: if ok to be backported
 >
 > Actually, the test tests for the existance of both maps:
 >
 > {{{
 > if ((cellinp != NULL) && (cellout != NULL)) {
 > }}}
 >
 > so the message will pop up if either of them is missing or if both are
 missing, so this is not very helpful. Especially when you don't give
 either of them, you get:
 >
 >
 > {{{
 > ATTENTION: Unable to create <(null)> raster map without cross_input
 raster
 >            map being specified
 > }}}
 >
 > I think this should be handled by parameter dependency rules for the
 parser, and not by the code. Something like:
 >
 > {{{
 > G_option_collective(parm.cellinp, parm.cellout)
 > }}}
 >

 Try the attached patch which implements the above and other checks that
 were done in the code.

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/3100#comment:7>
GRASS GIS <https://grass.osgeo.org>



More information about the grass-dev mailing list