Skip to content

Conversation

@ttnghia
Copy link
Contributor

@ttnghia ttnghia commented Nov 25, 2025

This stops creating a nullmask during the computation of groupby aggregation M2, similar to #20716. Such nullmask is unnecessary as M2 values are used together with valid count which already contains enough information for further computation (for std/variance).

@ttnghia ttnghia self-assigned this Nov 25, 2025
@ttnghia ttnghia requested a review from a team as a code owner November 25, 2025 04:07
@ttnghia ttnghia requested a review from bdice November 25, 2025 04:07
@ttnghia ttnghia added this to libcudf Nov 25, 2025
@ttnghia ttnghia added the 3 - Ready for Review Ready for review by team label Nov 25, 2025
@ttnghia ttnghia requested a review from karthikeyann November 25, 2025 04:07
@ttnghia ttnghia added libcudf Affects libcudf (C++/CUDA) code. Performance Performance related issue Spark Functionality that helps Spark RAPIDS improvement Improvement / enhancement to an existing function breaking Breaking change labels Nov 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3 - Ready for Review Ready for review by team breaking Breaking change improvement Improvement / enhancement to an existing function libcudf Affects libcudf (C++/CUDA) code. Performance Performance related issue Spark Functionality that helps Spark RAPIDS

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

1 participant