Skip to content

Commit

Permalink
Add the missing num_aggregations axis for groupby_max_cardinality (
Browse files Browse the repository at this point in the history
…#16630)

This PR fixes a minor bug where the `num_aggregations` axis was missed when working on #16154.

Authors:
  - Yunsong Wang (https://github.com/PointKernel)

Approvers:
  - Bradley Dice (https://github.com/bdice)
  - David Wendt (https://github.com/davidwendt)

URL: #16630
  • Loading branch information
PointKernel authored Aug 23, 2024
1 parent 8d6b261 commit a7ca3af
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions cpp/benchmarks/groupby/group_max.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -101,4 +101,5 @@ NVBENCH_BENCH_TYPES(bench_groupby_max,

NVBENCH_BENCH_TYPES(bench_groupby_max_cardinality, NVBENCH_TYPE_AXES(nvbench::type_list<int32_t>))
.set_name("groupby_max_cardinality")
.add_int64_axis("num_aggregations", {1})
.add_int64_axis("cardinality", {10, 20, 50, 100, 1'000, 10'000, 100'000, 1'000'000, 10'000'000});

0 comments on commit a7ca3af

Please sign in to comment.