-
Notifications
You must be signed in to change notification settings - Fork 912
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.
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
Support dask>=2024.11.2 in Dask cuDF #17439
Support dask>=2024.11.2 in Dask cuDF #17439
Conversation
|
||
This is an alias for `Series.dtype`. | ||
""" | ||
return self.dtype |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Turns out that Pandas uses Series.dtypes
as an alias for Series.dtype
(and some recent dask-expr code takes advantage of the Sereies<->DataFrame portability of the dtypes
attribute).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left one minor suggestion, otherwise as long as CI passes it looks good. Thanks @rjzamora !
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The cudf changes look good.
/merge |
Unpins the `dask`, `dask-expr`, and `distributed` versions for 25.02 development. Depends on rapidsai/cudf#17439 (**Update**: Now merged). Authors: - Richard (Rick) Zamora (https://github.com/rjzamora) Approvers: - Peter Andreas Entschev (https://github.com/pentschev) URL: #72
Description
This updates Dask cuDF so that we can unpin from 2024.11.2 for 25.02 development in
TODO: Create a branch in rapids-dask-dependency to unpin from 2024.11.2 (to demonstrate that CI passes).
Checklist