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

Add force_polygon_cw and force_polygon_ccw expression functions #45718

Merged
merged 1 commit into from Nov 1, 2021

Conversation

nyalldawson
Copy link
Collaborator

@nyalldawson nyalldawson commented Oct 29, 2021

These are recommended for use instead of the existing force_rhr function, due to the variability in definition of the "right hand rule" between different software applications. Using an explicit force_polygon_cw/ccw function removes user confusion when the results vary between different applications.

(Note that this matches PostGIS' recommendations now: see https://postgis.net/docs/ST_ForceRHR.html )

I'd like to backport this to 3.22 too.

@nyalldawson nyalldawson added Expressions Related to the QGIS expression engine or specific expression functions Needs Documentation When merging a labeled PR, an issue will be created in the Doc repo. labels Oct 29, 2021
@github-actions
Copy link

@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!

@github-actions github-actions bot added this to the 3.24.0 milestone Oct 29, 2021
These are recommended for use instead of the existing force_rhr
function, due to the variability in definition of the "right hand
rule" between different software applications. Using an explicit
force_polygon_cw/ccw function removes user confusion when the
results vary between different applications.
@github-actions
Copy link

github-actions bot commented Nov 1, 2021

@nyalldawson
A documentation ticket has been opened at qgis/QGIS-Documentation#7010
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!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Expressions Related to the QGIS expression engine or specific expression functions Needs Documentation When merging a labeled PR, an issue will be created in the Doc repo.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants