Skip to content
This repository was archived by the owner on Jul 28, 2025. It is now read-only.

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

Merged
GPUtester merged 1 commit intobranch-23.08from
branch-23.06
May 31, 2023
Merged

Forward-merge branch-23.06 to branch-23.08#1159
GPUtester merged 1 commit intobranch-23.08from
branch-23.06

Conversation

@GPUtester
Copy link
Copy Markdown
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 isVoid and trxcllnt May 31, 2023 13:08
@GPUtester GPUtester merged commit 085d946 into branch-23.08 May 31, 2023
@GPUtester
Copy link
Copy Markdown
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 subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

libcuspatial Relates to the cuSpatial C++ library

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

2 participants