Bug report #14912

'Check Geometries' plugin fails to fix duplicated nodes

Added by R. R. almost 4 years ago. Updated almost 4 years ago.

Status:Closed
Priority:Normal
Assignee:Sandro Mani
Category:C++ plugins/Geometry Checker
Affected QGIS version:master Regression?:No
Operating System: Easy fix?:No
Pull Request or Patch supplied:No Resolution:
Crashes QGIS or corrupts data:No Copied to github as #:22865

Description

Please try to fix the attached polygon. The geometry check ignores the duplicated nodes and doesn't return any results.

See also #14890

14912.zip (6.58 KB) R. R., 2016-05-27 01:33 PM

14912.mp4 - screencast (2.58 MB) R. R., 2016-05-27 01:52 PM

History

#1 Updated by R. R. almost 4 years ago

#2 Updated by Sandro Mani almost 4 years ago

  • Status changed from Open to Closed

The plugin currently only searches for consecutive duplicate nodes, which is not the case in the attached geometry. I believe such spikes as in your geometry are better detected with the minimal angle test. Checking all nodes against all other nodes in the geometry can quickly become prohibitive in terms of performance, plus such non-consecutive duplicate nodes are not per-se invalid, and may also be desiderable in some circumstances.

Feel free to reopen if you disagree or if I missed something.

#3 Updated by R. R. almost 4 years ago

Sandro, thanks for your reply. The minimum angle check works just fine.

Also available in: Atom PDF