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

Add fusedL2NN benchmark #936

Merged
merged 3 commits into from
Oct 27, 2022
Merged

Add fusedL2NN benchmark #936

merged 3 commits into from
Oct 27, 2022

Conversation

Nyrio
Copy link
Contributor

@Nyrio Nyrio commented Oct 21, 2022

Adds a benchmark for fusedL2NN. The benchmark used the wrapper fusedL2NNMinReduce compiled in the distance library, for faster compilation times.

@Nyrio Nyrio requested review from a team as code owners October 21, 2022 12:07
@Nyrio Nyrio added 3 - Ready for Review improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Oct 21, 2022
@Nyrio Nyrio marked this pull request as draft October 25, 2022 11:48
@Nyrio
Copy link
Contributor Author

Nyrio commented Oct 25, 2022

Marking as draft because I realized a fusedL2NN benchmark exists under cpp/bench/neighbors

@Nyrio Nyrio added 2 - In Progress Currenty a work in progress and removed 3 - Ready for Review labels Oct 25, 2022
@Nyrio
Remove former fusedL2NN benchmark under cpp/bench/neighbors, add cust…
44ec5e1 
…om counters, remove invalid but unused kernel policies
@Nyrio Nyrio marked this pull request as ready for review October 25, 2022 15:01
@Nyrio Nyrio added 3 - Ready for Review and removed 2 - In Progress Currenty a work in progress labels Oct 25, 2022
@cjnolet
Copy link
Member

cjnolet commented Oct 27, 2022

@gpucibot merge

@rapids-bot rapids-bot bot merged commit 93a20b8 into rapidsai:branch-22.12 Oct 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cjnolet cjnolet cjnolet approved these changes

Assignees
No one assigned
Labels
3 - Ready for Review CMake cpp improvement Improvement / enhancement to an existing function non-breaking Non-breaking change
Projects
VS/ML/DM Primitives Release Board
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Nyrio @cjnolet