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

Fix IsSimpleOp to support FindAllLocations for all geometry types #754

Merged
merged 6 commits into from
Jul 6, 2021

Conversation

FObermaier
Copy link
Contributor

@FObermaier FObermaier commented Jul 2, 2021

... for

  • Polygon,
  • MultiPoint and
  • GeometryCollection

geometries.
fixes #753

Signed-off-by: Felix Obermaier felix.obermaier@netcologne.de

... for `MultiPoint` geometries

fixes locationtech#753

Signed-off-by: Felix Obermaier <felix.obermaier@netcologne.de>
refers to locationtech#753

Signed-off-by: Felix Obermaier <felix.obermaier@netcologne.de>
refers to locationtech#753

Signed-off-by: Felix Obermaier <felix.obermaier@netcologne.de>
refers to locationtech#753

Signed-off-by: Felix Obermaier <felix.obermaier@netcologne.de>
refers to locationtech#753

Signed-off-by: Felix Obermaier <felix.obermaier@netcologne.de>
@dr-jts
Copy link
Contributor

dr-jts commented Jul 5, 2021

@FObermaier This is a nice enhancement - thanks!

@dr-jts dr-jts changed the title Fix new IsSimpleOp with isFindAllLocations=true Fix IsSimpleOp to support isFindAllLocations for all geometry types Jul 5, 2021
@dr-jts dr-jts changed the title Fix IsSimpleOp to support isFindAllLocations for all geometry types Fix IsSimpleOp to support FindAllLocations for all geometry types Jul 5, 2021
refers to locationtech#753

Signed-off-by: Felix Obermaier <felix.obermaier@netcologne.de>
@dr-jts dr-jts merged commit 6782b2e into locationtech:master Jul 6, 2021
pramsey added a commit to pramsey/geos that referenced this pull request Jul 6, 2021
strk pushed a commit to libgeos/geos that referenced this pull request Jul 6, 2021
@FObermaier FObermaier deleted the fix/IsSimpleOp_MultiPoint branch July 7, 2021 05:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

IsSimpleOp does not support FindAllLocations for Polygon, MultiPoint and GeometryCollection
2 participants