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

Use Geometry rather than MultiPolygon for indexing records #5615

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

jodygarnett
Copy link
Contributor

This is an experiment to try using Geometry (rather than MultiPolygon) in GeomUtils methods. Now that the index is no longer limited to MultiPolygons we should be able to work with any geometry content.

See previous discussion on #5396 with @fxprunayre

This affects parseGml and getSpatialExtent which now return a Geometry (rather than a MultiPolygon).
@jodygarnett
Copy link
Contributor Author

@fxprunayre if you are happy with this to can replace #5396

@jodygarnett jodygarnett changed the title To geometry Use Geometry rather than MultiPolygon for indexing Apr 21, 2021
@jodygarnett jodygarnett changed the title Use Geometry rather than MultiPolygon for indexing Use Geometry rather than MultiPolygon for indexing records Apr 21, 2021
@fxprunayre fxprunayre added this to the 4.0.5 milestone May 5, 2021
@fxprunayre fxprunayre modified the milestones: 4.0.5, 4.0.6 Jun 18, 2021
@jahow jahow modified the milestones: 4.0.6, 4.0.7 Feb 2, 2022
@fxprunayre fxprunayre modified the milestones: 4.2.0, 4.2.1 May 19, 2022
@fxprunayre fxprunayre modified the milestones: 4.2.1, 4.2.2 Sep 12, 2022
@fxprunayre fxprunayre modified the milestones: 4.2.2, 4.2.3 Dec 12, 2022
@fxprunayre fxprunayre modified the milestones: 4.2.3, 4.2.4 Mar 7, 2023
@fxprunayre fxprunayre modified the milestones: 4.2.4, 4.2.5 May 10, 2023
@fxprunayre fxprunayre modified the milestones: 4.2.5, 4.2.6 Jul 5, 2023
@fxprunayre fxprunayre modified the milestones: 4.2.6, 4.4.1 Oct 4, 2023
@fxprunayre fxprunayre modified the milestones: 4.4.1, 4.4.2 Nov 22, 2023
@fxprunayre fxprunayre modified the milestones: 4.4.2, 4.4.3 Jan 23, 2024
@fxprunayre fxprunayre removed this from the 4.4.3 milestone Mar 13, 2024
@fxprunayre fxprunayre added this to the 4.4.4 milestone Mar 13, 2024
@fxprunayre fxprunayre modified the milestones: 4.4.4, 4.4.5 Apr 16, 2024
@fxprunayre fxprunayre modified the milestones: 4.4.5, 4.4.6 Jun 4, 2024
@fxprunayre fxprunayre modified the milestones: 4.4.6, 4.4.7 Oct 15, 2024
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

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

Successfully merging this pull request may close these issues.

4 participants