Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[FEA] stddevsamp function #62

Closed
3 tasks done
revans2 opened this issue May 29, 2020 · 4 comments · Fixed by #8207
Closed
3 tasks done

[FEA] stddevsamp function #62

revans2 opened this issue May 29, 2020 · 4 comments · Fixed by #8207
Assignees
@ttnghia
Labels
cudf_dependency An issue or PR with this label depends on a new feature in cudf feature request New feature or request SQL part of the SQL/Dataframe plugin

Comments

@revans2
Copy link
Collaborator

revans2 commented May 29, 2020 •
edited by ttnghia
Loading

Is your feature request related to a problem? Please describe.
It would be great to be able to support the stddev_samp, std, stddev functions (same implementation)

The issue is that this is not trivial. Not sure if it is something we can implement out of the box, or if we are going to need some help from cudf.

We need to support these operations for the following contexts:
@revans2 revans2 added feature request New feature or request ? - Needs Triage Need team to review and classify SQL part of the SQL/Dataframe plugin labels May 29, 2020
@sameerz sameerz removed the ? - Needs Triage Need team to review and classify label Sep 1, 2020
wjxiz1992 pushed a commit to wjxiz1992/spark-rapids that referenced this issue Oct 29, 2020
@GaryShen2008 @krajendrannv
Update the pic for EMR "emr-5.23.0" to "emr-5.27.0" (NVIDIA#62)
81528e5
@revans2 revans2 added the cudf_dependency An issue or PR with this label depends on a new feature in cudf label Feb 18, 2021
@razajafri
Copy link
Collaborator

Should this be closed as part of #3373

@ttnghia
Copy link
Collaborator

ttnghia commented Oct 13, 2021

Should this be closed as part of #3373

No, as this issue should has 3 tasks while #3373 is just the first task of this.

@ttnghia ttnghia mentioned this issue Mar 28, 2023
Closed
@ttnghia
Copy link
Collaborator

ttnghia commented Mar 28, 2023

The second task should also be addressed by #3869.

@ttnghia ttnghia mentioned this issue Apr 14, 2023
Closed
@ttnghia ttnghia self-assigned this Apr 14, 2023
@ttnghia
Copy link
Collaborator

ttnghia commented Apr 14, 2023 •
edited
Loading

I've a cudf dependency issue for the 3rd item: rapidsai/cudf#13136
Edit: The 3rd item no longer depends on cudf.

@ttnghia ttnghia mentioned this issue Apr 28, 2023
Merged
tgravescs pushed a commit to tgravescs/spark-rapids that referenced this issue Nov 30, 2023
@nvauto
Update submodule cudf to ea2508e (NVIDIA#62)
bea847e 
Signed-off-by: spark-rapids automation <[email protected]>
binmahone pushed a commit to binmahone/spark-rapids that referenced this issue Mar 12, 2025
@winningsix
[auto-merge] automerge to main [bot] (NVIDIA#62)
dcf55f9 
auto-merge triggered by CI on `automerge` to create a PR keeping `main`
up-to-date.

**IMPORTANT:** 
- Before merging this PR, be sure to change the merging strategy to
`Create a merge commit` (repo admin only).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ttnghia ttnghia

Labels
cudf_dependency An issue or PR with this label depends on a new feature in cudf feature request New feature or request SQL part of the SQL/Dataframe plugin
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Support std aggregation in reduction ttnghia/spark-rapids
4 participants
@revans2 @sameerz @ttnghia @razajafri