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

IsSimple incorrectly reports duplicate line elements as simple #716

Closed
dr-jts opened this issue May 12, 2021 · 1 comment · Fixed by #717
Closed

IsSimple incorrectly reports duplicate line elements as simple #716

dr-jts opened this issue May 12, 2021 · 1 comment · Fixed by #717
Assignees

Comments

@dr-jts
Copy link
Contributor

dr-jts commented May 12, 2021

According to the OGC SFS specification,

A MultiCurve is simple if and only if all of its elements are simple and the only intersections between any two elements occur at Points that are on the boundaries of both elements.

This means that MultiLineString containing two elements with coincident line segments is not simple, since the elements intersect in their interior points.

MULTILINESTRING ((0 0, 100 100), (100 100, 0 0))

This is reported incorrectly as simple by Geometry.isSimple() and IsSimpleOp.

@dr-jts
Copy link
Contributor Author

dr-jts commented May 12, 2021

A new implementation of IsSimpleOp is under development, and will fix this problem (as well as being faster and simpler).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant