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

Simplify header-only point_distance API, support multipoint #704

Closed
Tracked by #703
isVoid opened this issue Sep 29, 2022 · 1 comment · Fixed by #731 or #734
Closed
Tracked by #703

Simplify header-only point_distance API, support multipoint #704

isVoid opened this issue Sep 29, 2022 · 1 comment · Fixed by #731 or #734
Assignees
Labels
improvement Improvement / enhancement to an existing function libcuspatial Relates to the cuSpatial C++ library

Comments

@isVoid
Copy link
Contributor

isVoid commented Sep 29, 2022

Header only Point distance API should be updated to support multipoint to multipoint distance. Adopt iterator collection structure to simplify the interface.

@isVoid isVoid added improvement Improvement / enhancement to an existing function c++ labels Sep 29, 2022
@isVoid isVoid self-assigned this Sep 29, 2022
@isVoid
Copy link
Contributor Author

isVoid commented Sep 29, 2022

Depends on #703

@isVoid isVoid added this to cuSpatial Oct 3, 2022
@harrism harrism moved this to Todo in cuSpatial Oct 3, 2022
@rapids-bot rapids-bot bot closed this as completed in #731 Oct 19, 2022
rapids-bot bot pushed a commit that referenced this issue Oct 19, 2022

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
…I to support multipoint to multipoint distance. (#731)

closes #704

Contributes to #703 

This PR introduces `Multipoint_range` interface, and simplifies the API of `point_distance`. Also updates the `point_distance` to support multipoint-multipoint distance.

Authors:
  - Michael Wang (https://github.com/isVoid)

Approvers:
  - H. Thomson Comer (https://github.com/thomcom)
  - Mark Harris (https://github.com/harrism)

URL: #731
Repository owner moved this from Todo to Done in cuSpatial Oct 19, 2022
@isVoid isVoid linked a pull request Oct 19, 2022 that will close this issue
3 tasks
@harrism harrism added libcuspatial Relates to the cuSpatial C++ library and removed c++ labels Nov 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement Improvement / enhancement to an existing function libcuspatial Relates to the cuSpatial C++ library
Projects
Status: Done
2 participants