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.

Sign up for GitHub

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

Jump to bottom

[SEDONA-484] [SEDONA-492] Implement ST_IsPolygonCW and ST_ForcePolygonCW #1286

Merged
merged 13 commits into from
Mar 26, 2024
Merged

[SEDONA-484] [SEDONA-492] Implement ST_IsPolygonCW and ST_ForcePolygonCW #1286

merged 13 commits into from
Mar 26, 2024

Conversation

furqaankhan
Copy link
Contributor

Did you read the Contributor Guide?

Is this PR related to a JIRA ticket?

What changes were proposed in this PR?

  • Add ST_IsPolygonCW
  • Add ST_ForcePolygonCW

How was this patch tested?

  • Passed new and old tests

Did this PR include necessary documentation updates?

  • Yes, I have updated the documentation update.

@furqaankhan furqaankhan marked this pull request as draft March 22, 2024 01:12
@furqaankhan furqaankhan marked this pull request as ready for review March 22, 2024 17:19
@jiayuasu
Copy link
Member

@furqaankhan Please resolve the conflict. Thanks!

@jiayuasu jiayuasu added this to the sedona-1.6.0 milestone Mar 26, 2024
@jiayuasu jiayuasu merged commit 7a49ef7 into apache:master Mar 26, 2024
37 of 49 checks passed
@furqaankhan furqaankhan deleted the st-force-polygon-cw branch July 16, 2024 01:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jiayuasu jiayuasu jiayuasu approved these changes

Assignees
No one assigned
Labels
affect public APIs improvement sedona-common sedona-spark
Projects
None yet
Milestone
sedona-1.6.0
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@furqaankhan @jiayuasu