Skip to content

Commit

Permalink
Remove assertion of post-conditions on groupby helper
Browse files Browse the repository at this point in the history
These invariants are guaranteed by the constructor.
  • Loading branch information
wence- committed Dec 18, 2024
1 parent fcf7c34 commit 3d3e80b
Showing 1 changed file with 0 additions and 4 deletions.
4 changes: 0 additions & 4 deletions cpp/src/rolling/grouped_rolling.cu
Original file line number Diff line number Diff line change
Expand Up @@ -208,10 +208,6 @@ std::unique_ptr<column> grouped_rolling_window(table_view const& group_keys,
// groups.)
// 3. [0, 500, 1000] indicates two equal-sized groups: [0,500), and [500,1000).

assert(group_offsets.size() >= 2 && group_offsets.element(0, stream) == 0 &&
group_offsets.element(group_offsets.size() - 1, stream) == input.size() &&
"Must have at least one group.");

if (aggr.kind == aggregation::CUDA || aggr.kind == aggregation::PTX) {
cudf::detail::preceding_window_wrapper grouped_preceding_window{
group_offsets.data(), group_labels.data(), preceding_window};
Expand Down

0 comments on commit 3d3e80b

Please sign in to comment.