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

Forward-merge branch-23.06 to branch-23.08 #1159

Merged
merged 1 commit into from
May 31, 2023
Merged

Conversation

GPUtester
Copy link
Contributor

Forward-merge triggered by push to branch-23.06 that creates a PR to keep branch-23.08 up-to-date. If this PR is unable to be immediately merged due to conflicts, it will remain open for the team to manually merge.

This PR refactors `ST_Distance` API to reduce the number of kernels to maintain. Currently, each st_distance API maintains its own distance kernel. This refactor let linestring_polygon distance and polygon_polygon distance share the underlying linestring-linestring distance. Also, point-polygon distance now share the same kernel with point-linestring distance kernel.

As we are moving to optimization, reducing the total number of kernel to maintain can help scaling the optimization benefit across multiple APIs.

Authors:
  - Michael Wang (https://github.com/isVoid)
  - Paul Taylor (https://github.com/trxcllnt)
  - Mark Harris (https://github.com/harrism)

Approvers:
  - Mark Harris (https://github.com/harrism)

URL: #1143
@GPUtester GPUtester requested a review from a team as a code owner May 31, 2023 13:08
@GPUtester GPUtester requested review from trxcllnt and isVoid May 31, 2023 13:08
@GPUtester GPUtester merged commit 085d946 into branch-23.08 May 31, 2023
@GPUtester
Copy link
Contributor Author

SUCCESS - forward-merge complete.

@github-actions github-actions bot added the libcuspatial Relates to the cuSpatial C++ library label May 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
libcuspatial Relates to the cuSpatial C++ library
Projects
Status: Todo
Development

Successfully merging this pull request may close these issues.

2 participants