Expose GEOS coverage simplify as a processing algorithm

[nyalldawson]

Exposes the new (in geos v12) coverage simplification tool as a processing algorithm.

Open questions:

• should we force validation of the coverage prior to simplification? It's added slowness, but the algorithm does requires valid coverages.
• Should the existing geometry validation check apply to coverages also? Ie we check both the feature geometry AND the coverage geometry?

Peek.2023-10-16.11-38.mp4

[pathmapper]

should we force validation of the coverage prior to simplification?

+1

Should the existing geometry validation check apply to coverages also? Ie we check both the feature geometry AND the coverage geometry?

-1

I'm afraid of a failing check because of the coverage for use cases where only a valid geometry is needed...

What about a dedicated coverage validation option for the Check validity processing tool?

[nyalldawson]

@pathmapper

I'm afraid of a failing check because of the coverage for use cases where only a valid geometry is needed...

Ah, I think you misunderstood. I was referring specifically to this algorithm only, making it do a forced coverage validity check if the geometry check setting is anything other then "ignore". Other non-coverage related algorithms wouldn't be changed.

What about a dedicated coverage validation option for the Check validity processing tool?

It's on its way! (I just cant get the geos coverage check method to work correctly right now 😳)

[pathmapper]

Thanks, got it :-) Then I'm +1 to check both the feature geometry AND the coverage geometry if the geometry check setting is anything other then "ignore".

It's on its way!

👍

[qgis-bot]

@nyalldawson

This pull request has been tagged for the changelog.

• The description will be harvested so please provide a "nearly-ready" text for the final changelog
• If possible, add a nice illustration of the feature. Only the first one in the description will be harvested (GIF accepted as well)
• If you can, it's better to give credits to your sponsor, see below for different formats.

You can edit the description.

Format available for credits

• Funded by NAME
• Funded by URL
• Funded by NAME URL
• Sponsored by NAME
• Sponsored by URL
• Sponsored by NAME URL

Thank you!

[qgis-bot]

@nyalldawson
This pull request has been tagged as requiring documentation.

A documentation ticket will be opened at https://github.com/qgis/QGIS-Documentation when this PR is merged.

Please update the description (not the comments) with helpful description and screenshot to help the work from documentors.
Also, any commit having [needs-doc] or [Needs Documentation] in will see its message pushed to the issue, so please be as verbose as you can.

Thank you!

[qgis-bot]

@nyalldawson
A documentation ticket has been opened at qgis/QGIS-Documentation#9082
It is your responsibility to visit this ticket and add as much detail as possible for the documentation team to correctly document this change.
Thank you!