[GRASS-dev] [GRASS GIS] #647: v.generalize's remove_small deletes wrong boundary

#647: v.generalize's remove_small deletes wrong boundary
---------------------+------------------------------------------------------
Reporter: hoeth | Owner: grass-dev@lists.osgeo.org
     Type: defect | Status: new
Priority: normal | Milestone: 6.4.0
Component: default | Version: unspecified
Keywords: | Platform: Linux
      Cpu: x86-64 |
---------------------+------------------------------------------------------
Running v.generalize method=remove_small on the boundaries in
"working.ascii" and "broken.ascii" yields different results, even though
the only difference between the two files is the order in which the
boundaries are stored. For "working.ascii" only the small open boundary is
removed, while for "broken.ascii" both boundaries are removed.

I observe this behaviour both in 6.3.0 and in 6.4.0RC5 (didn't try any
other releases).

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

#647: v.generalize's remove_small deletes wrong boundary
---------------------+------------------------------------------------------
  Reporter: hoeth | Owner: grass-dev@lists.osgeo.org
      Type: defect | Status: new
  Priority: normal | Milestone: 6.4.0
Component: Vector | Version: 6.4.0 RCs
Resolution: | Keywords: v.generalize
  Platform: Linux | Cpu: x86-64
---------------------+------------------------------------------------------
Changes (by hamish):

  * keywords: => v.generalize
  * version: unspecified => 6.4.0 RCs
  * component: default => Vector

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

#647: v.generalize's remove_small deletes wrong boundary
--------------------------+-------------------------------------------------
Reporter: hoeth | Owner: grass-dev@…
     Type: defect | Status: new
Priority: normal | Milestone: 6.4.0
Component: Vector | Version: 6.4.0 RCs
Keywords: v.generalize | Platform: Linux
      Cpu: x86-64 |
--------------------------+-------------------------------------------------

Comment(by mmetz):

The `remove_small` method has been removed because it was faulty. Working
equivalents (that is, the original cleaning functions) are as before
provided by `v.clean`.

Markus M

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

#647: v.generalize's remove_small deletes wrong boundary
---------------------+------------------------------------------------------
  Reporter: hoeth | Owner: grass-dev@…
      Type: defect | Status: closed
  Priority: normal | Milestone: 6.4.0
Component: Vector | Version: 6.4.0 RCs
Resolution: fixed | Keywords: v.generalize
  Platform: Linux | Cpu: x86-64
---------------------+------------------------------------------------------
Changes (by mmetz):

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

Comment:

`v.generalize` has been rewritten and fixed in all active branches.
Closing as fixed.

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