#449 closed defect (fixed)
Assertion fails checking validity of polygon
Reported by: | strk | Owned by: | strk |
---|---|---|---|
Priority: | blocker | Milestone: | 3.4.0 |
Component: | Default | Version: | master |
Severity: | Critical | Keywords: | |
Cc: |
Description
See http://trac.osgeo.org/postgis/ticket/1007 for the dataset.
bool geos::operation::valid::IndexedNestedRingTester::isNonNested(): Assertion `innerRingPt!=null' failed.
Attachments (2)
Change History (16)
comment:1 Changed 10 years ago by
Owner: | changed from geos-devel@… to strk |
---|---|
Status: | new → assigned |
Changed 10 years ago by
Attachment: | bug449.xml added |
---|
comment:2 Changed 10 years ago by
comment:3 Changed 10 years ago by
The message is correct: a triangle hole has all vertices touching nearby holes. I'll have to dig further in order to tell if that's an invalidity of input (sounds like )
Changed 10 years ago by
Attachment: | bug449-simp.xml added |
---|
comment:4 Changed 10 years ago by
The bug449-simp.xml file is an extremely simplified version of the test. It's a polygon with two holes. One of the two holes shares all vertices with another hole, thus triggering the assertion failure. I pretty much belive this case being invalid, and I'm trying to think of a possible other _valid_ case. I dubt I'll find one though..
comment:5 Changed 10 years ago by
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Fixed on the GEOS side with r3392. The changeset includes a proper XML (the one attached here was malformed). JTS still fails.
comment:7 Changed 10 years ago by
Resolution: | fixed |
---|---|
Status: | closed → reopened |
JTS fixed this. It should be checked _how_ and make sure GEOS follows that.
comment:8 Changed 10 years ago by
Also, should be back-ported to 3.3 branch (trunk became 3.4.0 instead)
comment:9 Changed 10 years ago by
comment:10 Changed 10 years ago by
Milestone: | 3.3.1 → 3.4.0 |
---|---|
Version: | 3.3.0 → svn-trunk |
#456 is the backport to 3.3 branch
comment:11 Changed 10 years ago by
Resolution: | → fixed |
---|---|
Status: | reopened → closed |
Note: JTS (rev 398) also fails against the attached testcase: