Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

"Check Geometry Validity" produces incorrect result #20830

Closed
qgib opened this issue May 11, 2015 · 6 comments
Closed

"Check Geometry Validity" produces incorrect result #20830

qgib opened this issue May 11, 2015 · 6 comments
Labels
Bug Either a bug report, or a bug fix. Let's hope for the latter! High Priority

Comments

@qgib
Copy link
Contributor

qgib commented May 11, 2015

Author Name: Daniel Baston (@dbaston)
Original Redmine Issue: 12730
Affected QGIS version: 2.8.1


The following is a valid, two-part MULTIPOLYGON:

MULTIPOLYGON (((-71.757875 44.565735, -71.748526 44.560716, -71.758014 44.564355, -71.757875 44.565735)),
((-71.760777 44.567292, -71.75725 44.56739, -71.757875 44.565735, -71.760777 44.567292)))

The "Check Geometry Validity" tool reports the following:

1,segment 0 of ring 0 of polygon 0 intersects segment 0 of ring 1 of polygon 1 at -71.757875, 44.565735
1,Geometry has 1 errors.

Using build 2863f20

@qgib
Copy link
Contributor Author

qgib commented May 20, 2015

Author Name: Arnaud Morvan (@arnaud-morvan)


The two parts of the polygon have one point in common.
In Settings/Options/Digitizing/Feature creation, there is two options for validity checking which are QGIS or GEOS.
When checking the validity from ftools menu entry,
QGIS method effectively returns that the poligon is not valid:
1,segment 0 of ring 0 of polygon 0 intersects segment 0 of ring 1 of polygon 1 at -71.757875, 44.565735
1,Geometry has 1 errors.
GEOS method returns that the polygon is valid.

@qgib
Copy link
Contributor Author

qgib commented May 20, 2015

Author Name: Giovanni Manghi (@gioman)


Arnaud Morvan wrote:

GEOS method returns that the polygon is valid.

shoud we close this then?


  • status_id was changed from Open to Feedback
  • category_id was configured as 44

@qgib
Copy link
Contributor Author

qgib commented May 21, 2015

Author Name: Daniel Baston (@dbaston)


That's interesting. Is the intent of the "QGIS Method" to validate against a different set of rules than GEOS / OGC?

@qgib
Copy link
Contributor Author

qgib commented Jun 23, 2015

Author Name: Daniel Baston (@dbaston)


Related to qgis/QGIS-Documentation#472


  • assigned_to_id was configured as Jürgen Fischer

@qgib
Copy link
Contributor Author

qgib commented Jun 23, 2015

Author Name: Jürgen Fischer (@jef-n)


  • category_id removed 44
  • assigned_to_id removed Jürgen Fischer

@qgib
Copy link
Contributor Author

qgib commented Jun 23, 2015

Author Name: Jürgen Fischer (@jef-n)


Fixed in changeset "3eef29ab1bd4cc007b7db5012dad23d9c39e2249".


  • status_id was changed from Feedback to Closed

@qgib qgib added Bug Either a bug report, or a bug fix. Let's hope for the latter! High Priority labels May 25, 2019
@qgib qgib closed this as completed May 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Either a bug report, or a bug fix. Let's hope for the latter! High Priority
Projects
None yet
Development

No branches or pull requests

1 participant