From a7ca3afb251805face3dd3248381f4cc9503e143 Mon Sep 17 00:00:00 2001 From: Yunsong Wang Date: Fri, 23 Aug 2024 12:24:30 -0700 Subject: [PATCH] Add the missing `num_aggregations` axis for `groupby_max_cardinality` (#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: https://github.com/rapidsai/cudf/pull/16630 --- cpp/benchmarks/groupby/group_max.cpp | 1 + 1 file changed, 1 insertion(+) diff --git a/cpp/benchmarks/groupby/group_max.cpp b/cpp/benchmarks/groupby/group_max.cpp index f41285008c4..b9a701a71f4 100644 --- a/cpp/benchmarks/groupby/group_max.cpp +++ b/cpp/benchmarks/groupby/group_max.cpp @@ -101,4 +101,5 @@ NVBENCH_BENCH_TYPES(bench_groupby_max, NVBENCH_BENCH_TYPES(bench_groupby_max_cardinality, NVBENCH_TYPE_AXES(nvbench::type_list)) .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});