Ticket #3619 (closed enhancement: fixed)

Opened 3 years ago

Last modified 3 years ago

Polygonize should have a connectedness parameter

Reported by: ajolma Owned by: ilucena
Priority: normal Milestone: 1.8.0
Component: Algorithms Version: svn-trunk
Severity: normal Keywords: polygonize
Cc:

Description

Currently polygonize defaults to 4-connectedness and there is no way to change that. I suggest we use the Options parameter for that. Attached is a tested diff which achieves just this.

Attachments

diff Download (1.5 KB) - added by ajolma 3 years ago.
diff of polygonize.cpp

Change History

Changed 3 years ago by ajolma

  • attachment diff Download added

diff of polygonize.cpp

  Changed 3 years ago by warmerdam

  • keywords polygonize added
  • owner changed from warmerdam to ilucena
  • version changed from unspecified to svn-trunk
  • component changed from default to Algorithms
  • milestone set to 1.8.0

Ivan,

please incorporate this patch and:

* Update the GDALPolygonize() docs to mention the new parameter. * Update the autotest/alg/polygonize.py test to include 8-connectedness testing.

This should only be in trunk for now I think.

  Changed 3 years ago by ilucena

  • status changed from new to closed
  • resolution set to fixed

All right. Patch applied on r19829

  Changed 3 years ago by warmerdam

  • status changed from closed to reopened
  • resolution fixed deleted

Ivan,

I think you neglected my notes about autotesting and documentation.

follow-up: ↓ 7   Changed 3 years ago by ilucena

I soon as re-start on Linux.

  Changed 3 years ago by ilucena

Doxygen changes on r19830

  Changed 3 years ago by ilucena

  • status changed from reopened to closed
  • resolution set to fixed

Autotest changes on 19831

in reply to: ↑ 4   Changed 3 years ago by ilucena

Replying to ilucena:

I soon as re-start on Linux.

Correction:

As soon as I re-start on Linux.

I shouldn't have closed.

Note: See TracTickets for help on using tickets.