Skip to content

Commit

Permalink
Fix cudf::rank not getting enough params (#16666)
Browse files Browse the repository at this point in the history
Fix issue #16624

Authors:
  - Jayjeet Chakraborty (https://github.com/JayjeetAtGithub)

Approvers:
  - Mark Harris (https://github.com/harrism)
  - Bradley Dice (https://github.com/bdice)

URL: #16666
  • Loading branch information
JayjeetAtGithub authored Aug 27, 2024
1 parent efa9770 commit f1cc962
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 0 deletions.
2 changes: 2 additions & 0 deletions cpp/benchmarks/sort/rank_lists.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -37,6 +37,8 @@ void nvbench_rank_lists(nvbench::state& state, nvbench::type_list<nvbench::enum_
cudf::order::ASCENDING,
null_frequency ? cudf::null_policy::INCLUDE : cudf::null_policy::EXCLUDE,
cudf::null_order::AFTER,
false,
cudf::get_default_stream(),
rmm::mr::get_current_device_resource());
});
}
Expand Down
2 changes: 2 additions & 0 deletions cpp/benchmarks/sort/rank_structs.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,8 @@ void nvbench_rank_structs(nvbench::state& state, nvbench::type_list<nvbench::enu
cudf::order::ASCENDING,
nulls ? cudf::null_policy::INCLUDE : cudf::null_policy::EXCLUDE,
cudf::null_order::AFTER,
false,
cudf::get_default_stream(),
rmm::mr::get_current_device_resource());
});
}
Expand Down

0 comments on commit f1cc962

Please sign in to comment.