Skip to content

Commit

Permalink
Fix missing parameter for select_k (#1682)
Browse files Browse the repository at this point in the history
The parameter `rmm::mr::device_memory_resource* mr` is missing while passing down.

Authors:
  - JiayuSun (https://github.com/ucassjy)

Approvers:
  - Ben Frederickson (https://github.com/benfred)

URL: #1682
  • Loading branch information
ucassjy authored Jul 28, 2023
1 parent bcb7404 commit 47a4385
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions cpp/include/raft/matrix/detail/select_k-inl.cuh
Original file line number Diff line number Diff line change
Expand Up @@ -303,7 +303,8 @@ void select_k(raft::resources const& handle,
out_idx,
select_min,
true, // fused_last_filter
stream);
stream,
mr);

if (sorted) {
auto offsets = raft::make_device_vector<IdxT, IdxT>(handle, (IdxT)(batch_size + 1));
Expand All @@ -326,7 +327,7 @@ void select_k(raft::resources const& handle,
case Algo::kWarpDistributedShm:
return detail::select::warpsort::
select_k_impl<T, IdxT, detail::select::warpsort::warp_sort_distributed_ext>(
in_val, in_idx, batch_size, len, k, out_val, out_idx, select_min, stream);
in_val, in_idx, batch_size, len, k, out_val, out_idx, select_min, stream, mr);
case Algo::kFaissBlockSelect:
return neighbors::detail::select_k(
in_val, in_idx, batch_size, len, out_val, out_idx, select_min, k, stream);
Expand Down

0 comments on commit 47a4385

Please sign in to comment.