[GRASS-dev] [GRASS GIS] #1789: v.hull: should not use

GRASS GIS GRASS GIS developer
1

#1789: v.hull: should not use
---------------------+------------------------------------------------------
Reporter: neteler | Owner: grass-dev@…
     Type: defect | Status: new
Priority: normal | Milestone: 6.4.3
Component: Vector | Version: svn-releasebranch64
Keywords: v.hull | Platform: All
      Cpu: All |
---------------------+------------------------------------------------------
If the current region is not set to the vector map, v.hull
fails with:

{{{
v.hull training_turkey_087VI174 out=training_turkey_087VI174_hull
ERROR: Convex hull calculation requires at least three points. Exiting.
}}}

The reason is in main.c, line 311:

{{{
     /* load site coordinates */
     G_get_window(&window);
     numSitePoints = loadSiteCoordinates(&Map, &points, all->answer,
&window);
}}}

...

In loadSiteCoordinates()
{{{
     /* copy window to box */
     Vect_region_box(window, &box);
}}}

I wonder why it is made region sensitive (untypical for GRASS
vector commands)...

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/1789&gt;
GRASS GIS <http://grass.osgeo.org>

2

#1789: v.hull: should not use
-------------------------+--------------------------------------------------
Reporter: neteler | Owner: grass-dev@…
     Type: enhancement | Status: new
Priority: normal | Milestone: 7.0.0
Component: Vector | Version: svn-releasebranch64
Keywords: v.hull | Platform: All
      Cpu: All |
-------------------------+--------------------------------------------------
Changes (by mmetz):

  * type: defect => enhancement
  * milestone: 6.4.3 => 7.0.0

Comment:

Replying to [ticket:1789 neteler]:
> If the current region is not set to the vector map, v.hull
> fails

In this case, you need to use the -a flag to ignore the current region.

I have changed the behaviour in trunk such that the current region is only
respected if the (new) -r flag is set (see also other v.in.* modules). The
-a flag has been removed.

Additionally, I have added selection options 'cats' and 'where'.

All changes are in r53759.

Markus M

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/1789#comment:1&gt;
GRASS GIS <http://grass.osgeo.org>

3

#1789: v.hull: should not use
--------------------------+-------------------------------------------------
  Reporter: neteler | Owner: grass-dev@…
      Type: enhancement | Status: closed
  Priority: normal | Milestone: 7.0.0
Component: Vector | Version: svn-releasebranch64
Resolution: fixed | Keywords: v.hull
  Platform: All | Cpu: All
--------------------------+-------------------------------------------------
Changes (by neteler):

  * status: new => closed
  * resolution: => fixed

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/1789#comment:2&gt;
GRASS GIS <http://grass.osgeo.org>

4

#1789: v.hull: should not be region sensitive
--------------------------+-------------------------------------------------
  Reporter: neteler | Owner: grass-dev@…
      Type: enhancement | Status: closed
  Priority: normal | Milestone: 7.0.0
Component: Vector | Version: svn-releasebranch64
Resolution: fixed | Keywords: v.hull
  Platform: All | Cpu: All
--------------------------+-------------------------------------------------

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/1789#comment:3&gt;
GRASS GIS <http://grass.osgeo.org>