diff --git a/cpp/src/rolling/grouped_rolling.cu b/cpp/src/rolling/grouped_rolling.cu index eee73d7b258..e5db808c9af 100644 --- a/cpp/src/rolling/grouped_rolling.cu +++ b/cpp/src/rolling/grouped_rolling.cu @@ -864,7 +864,7 @@ struct to_duration_bounds { } template (), void>* = nullptr> - range_window_bounds operator()(size_type num_days) const + range_window_bounds operator()(size_type) const { CUDF_FAIL("Expected timestamp orderby column."); } diff --git a/cpp/src/rolling/range_window_bounds_detail.hpp b/cpp/src/rolling/range_window_bounds_detail.hpp index 47c9273a232..e09e474d9de 100644 --- a/cpp/src/rolling/range_window_bounds_detail.hpp +++ b/cpp/src/rolling/range_window_bounds_detail.hpp @@ -79,6 +79,7 @@ namespace { template void assert_non_negative(T const& value) { + (void)value; if constexpr (std::numeric_limits::is_signed) { CUDF_EXPECTS(value >= T{0}, "Range scalar must be >= 0."); } diff --git a/cpp/src/rolling/rolling_detail.cuh b/cpp/src/rolling/rolling_detail.cuh index 525ed31ad82..715c72d4cb7 100644 --- a/cpp/src/rolling/rolling_detail.cuh +++ b/cpp/src/rolling/rolling_detail.cuh @@ -112,7 +112,7 @@ struct DeviceRolling { // perform the windowing operation template bool __device__ operator()(column_device_view const& input, - column_device_view const& ignored_default_outputs, + column_device_view const&, mutable_column_device_view& output, size_type start_index, size_type end_index, @@ -165,7 +165,7 @@ struct DeviceRollingArgMinMax { template bool __device__ operator()(column_device_view const& input, - column_device_view const& ignored_default_outputs, + column_device_view const&, mutable_column_device_view& output, size_type start_index, size_type end_index, @@ -218,7 +218,7 @@ struct DeviceRollingCountValid { template bool __device__ operator()(column_device_view const& input, - column_device_view const& ignored_default_outputs, + column_device_view const&, mutable_column_device_view& output, size_type start_index, size_type end_index, @@ -263,8 +263,8 @@ struct DeviceRollingCountAll { DeviceRollingCountAll(size_type _min_periods) : min_periods(_min_periods) {} template - bool __device__ operator()(column_device_view const& input, - column_device_view const& ignored_default_outputs, + bool __device__ operator()(column_device_view const&, + column_device_view const&, mutable_column_device_view& output, size_type start_index, size_type end_index, @@ -296,8 +296,8 @@ struct DeviceRollingRowNumber { DeviceRollingRowNumber(size_type _min_periods) : min_periods(_min_periods) {} template - bool __device__ operator()(column_device_view const& input, - column_device_view const& ignored_default_outputs, + bool __device__ operator()(column_device_view const&, + column_device_view const&, mutable_column_device_view& output, size_type start_index, size_type end_index, @@ -381,7 +381,7 @@ struct DeviceRollingLead { bool __device__ operator()(column_device_view const& input, column_device_view const& default_outputs, mutable_column_device_view& output, - size_type start_index, + size_type, size_type end_index, size_type current_index) { @@ -438,7 +438,7 @@ struct DeviceRollingLag { column_device_view const& default_outputs, mutable_column_device_view& output, size_type start_index, - size_type end_index, + size_type, size_type current_index) { // Offsets have already been normalized. @@ -531,7 +531,7 @@ struct create_rolling_operator< typename T = InputType, aggregation::Kind O = op, std::enable_if_t* = nullptr> - auto operator()(size_type min_periods, rolling_aggregation const& agg) + auto operator()(size_type min_periods, rolling_aggregation const&) { return typename corresponding_rolling_operator::type(min_periods); } @@ -539,7 +539,7 @@ struct create_rolling_operator< template * = nullptr> - auto operator()(size_type min_periods, rolling_aggregation const& agg) + auto operator()(size_type, rolling_aggregation const& agg) { return DeviceRollingLead{ dynamic_cast(agg).row_offset}; @@ -548,7 +548,7 @@ struct create_rolling_operator< template * = nullptr> - auto operator()(size_type min_periods, rolling_aggregation const& agg) + auto operator()(size_type, rolling_aggregation const& agg) { return DeviceRollingLag{ dynamic_cast(agg).row_offset}; @@ -595,7 +595,7 @@ class rolling_aggregation_preprocessor final : public cudf::detail::simple_aggre // Then a second pass uses those indices to gather the final strings. This step // translates the the MIN -> ARGMIN aggregation std::vector> visit(data_type col_type, - cudf::detail::min_aggregation const& agg) override + cudf::detail::min_aggregation const&) override { std::vector> aggs; aggs.push_back(col_type.id() == type_id::STRING ? make_argmin_aggregation() @@ -608,7 +608,7 @@ class rolling_aggregation_preprocessor final : public cudf::detail::simple_aggre // Then a second pass uses those indices to gather the final strings. This step // translates the the MAX -> ARGMAX aggregation std::vector> visit(data_type col_type, - cudf::detail::max_aggregation const& agg) override + cudf::detail::max_aggregation const&) override { std::vector> aggs; aggs.push_back(col_type.id() == type_id::STRING ? make_argmax_aggregation() @@ -619,7 +619,7 @@ class rolling_aggregation_preprocessor final : public cudf::detail::simple_aggre // COLLECT_LIST aggregations do not peform a rolling operation at all. They get processed // entirely in the finalize() step. std::vector> visit( - data_type col_type, cudf::detail::collect_list_aggregation const& agg) override + data_type, cudf::detail::collect_list_aggregation const&) override { return {}; } @@ -683,10 +683,10 @@ class rolling_aggregation_postprocessor final : public cudf::detail::aggregation } // all non-specialized aggregation types simply pass the intermediate result through. - void visit(aggregation const& agg) override { result = std::move(intermediate); } + void visit(aggregation const&) override { result = std::move(intermediate); } // perform a final gather on the generated ARGMIN data - void visit(cudf::detail::min_aggregation const& agg) override + void visit(cudf::detail::min_aggregation const&) override { if (result_type.id() == type_id::STRING) { // The rows that represent null elements will have negative values in gather map, @@ -704,7 +704,7 @@ class rolling_aggregation_postprocessor final : public cudf::detail::aggregation } // perform a final gather on the generated ARGMAX data - void visit(cudf::detail::max_aggregation const& agg) override + void visit(cudf::detail::max_aggregation const&) override { if (result_type.id() == type_id::STRING) { // The rows that represent null elements will have negative values in gather map, @@ -957,14 +957,14 @@ struct rolling_window_launcher { typename FollowingWindowIterator> std::enable_if_t::type::is_supported(), std::unique_ptr> - operator()(column_view const& input, - column_view const& default_outputs, - PrecedingWindowIterator preceding_window_begin, - FollowingWindowIterator following_window_begin, - int min_periods, - rolling_aggregation const& agg, - rmm::cuda_stream_view stream, - rmm::mr::device_memory_resource* mr) + operator()(column_view const&, + column_view const&, + PrecedingWindowIterator, + FollowingWindowIterator, + int, + rolling_aggregation const&, + rmm::cuda_stream_view, + rmm::mr::device_memory_resource*) { CUDF_FAIL("Invalid aggregation type/pair"); }