Skip to content

Generalize unique aggregate#7892

Merged
jakirkham merged 14 commits intodask:mainfrom
galipremsagar:7840
Jul 16, 2021
Merged

Generalize unique aggregate#7892
jakirkham merged 14 commits intodask:mainfrom
galipremsagar:7840

Conversation

@galipremsagar
Copy link
Copy Markdown
Contributor

@galipremsagar galipremsagar commented Jul 13, 2021

This PR generalizes the groupby unique aggregate operation so that other backends can correctly utilize this operation.

Co-authored-by: jakirkham <jakirkham@gmail.com>
@jrbourbeau
Copy link
Copy Markdown
Member

Just a heads up that these CI failures are unrelated to the changes here. Resolving in #7895

@jrbourbeau
Copy link
Copy Markdown
Member

Merged main to resolve the CI issues

@jakirkham jakirkham requested a review from rjzamora July 15, 2021 22:41
@galipremsagar
Copy link
Copy Markdown
Contributor Author

Can we merge this PR?

@jakirkham jakirkham merged commit b92c332 into dask:main Jul 16, 2021
@jakirkham
Copy link
Copy Markdown
Member

Yep thanks Prem! 😄

rapids-bot bot pushed a commit to rapidsai/cudf that referenced this pull request Jul 20, 2021
Fixes: #7840
Dependent on: dask/dask#7892

This PR introduces ability to construct `list` Series by passing in a sequence of array-like objects to `cudf.Series`.

Authors:
  - GALI PREM SAGAR (https://github.com/galipremsagar)

Approvers:
  - Richard (Rick) Zamora (https://github.com/rjzamora)
  - Charles Blackmon-Luca (https://github.com/charlesbluca)
  - Ashwin Srinath (https://github.com/shwina)

URL: #8735
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants