Opened 3 years ago

Closed 3 years ago

#1115 closed defect (fixed)

DouglasPeuckerSimplifier does not remove collapsed rings

Reported by: mdavis Owned by: mdavis
Priority: major Milestone: 3.9.2
Component: Core Version: 3.9.0
Severity: Significant Keywords:
Cc:

Description

DouglasPeuckerSimplifier is missing the JTS ​code to remove collapsed LinearRings.

This is a regression found during testing in ​R-sf issue 1655. It was surfaced by the enhancement made in ​e6c6fa39e

Change History (3)

comment:1 by mdavis, 3 years ago

Owner: changed from strk to mdavis

comment:2 by mdavis, 3 years ago

Reproducing case: 

MULTIPOLYGON (((-76.02716827 36.55671692, -75.99866486 36.55665207, -75.91191864 36.54253006, -75.92480469 36.47397614, -75.97727966 36.4780159, -75.97628784 36.51792526, -76.02716827 36.55671692)), ((-75.90198517 36.55619812, -75.8781662 36.55587387, -75.77315521 36.22925568, -75.78317261 36.22519302, -75.90198517 36.55619812)))

SimplifyDP(A, 0.05) should produce a Polygon, but currently produces GeometryCollection(Polygon, Line).

Last edited 3 years ago by mdavis (previous) (diff)

comment:3 by mdavis, 3 years ago

Resolution: → fixed
Status: new → closed

Fixed by ​f3dfbdc.

Note: See TracTickets for help on using tickets.