Skip to content

Fix sparse KNN for large batches#1640

Merged
rapids-bot[bot] merged 3 commits into
branch-23.08from
fix-sparse-knn-large-batches
Jul 26, 2023
Merged

Fix sparse KNN for large batches#1640
rapids-bot[bot] merged 3 commits into
branch-23.08from
fix-sparse-knn-large-batches

Conversation

@viclafargue

Copy link
Copy Markdown
Contributor

Answers #1187

@viclafargue viclafargue requested a review from a team as a code owner July 7, 2023 12:14
@github-actions github-actions Bot added the cpp label Jul 7, 2023

@tfeher tfeher left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @viclafargue for the PR, LGTM!

@tfeher

tfeher commented Jul 14, 2023

Copy link
Copy Markdown
Contributor

To resolve style errors, you could use pre-commit hooks

@codecov-commenter

codecov-commenter commented Jul 14, 2023

Copy link
Copy Markdown

Codecov Report

Patch and project coverage have no change.

Comparison is base (202385b) 87.98% compared to head (248dfe9) 87.98%.

Additional details and impacted files
@@              Coverage Diff              @@
##           branch-23.08    #1640   +/-   ##
=============================================
  Coverage         87.98%   87.98%           
=============================================
  Files                21       21           
  Lines               491      491           
=============================================
  Hits                432      432           
  Misses               59       59           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@cjnolet cjnolet added bug Something isn't working non-breaking Non-breaking change labels Jul 25, 2023
@cjnolet

cjnolet commented Jul 26, 2023

Copy link
Copy Markdown
Contributor

/merge

@rapids-bot rapids-bot Bot merged commit f25907b into branch-23.08 Jul 26, 2023
loulankxh pushed a commit to loulankxh/raft that referenced this pull request Oct 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3 - Ready for Review bug Something isn't working cpp non-breaking Non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants