-
Notifications
You must be signed in to change notification settings - Fork 920
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
Update 10 minutes to cuDF #12114
Update 10 minutes to cuDF #12114
Conversation
Closes rapidsai#4665 Closes rapidsai#4946 Closes rapidsai#10614 Closes rapidsai#11931
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## branch-23.02 #12114 +/- ##
===============================================
Coverage ? 88.26%
===============================================
Files ? 137
Lines ? 22586
Branches ? 0
===============================================
Hits ? 19935
Misses ? 2651
Partials ? 0 Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
Think I addressed all the comments. |
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.
Thanks @wence- for updating the notebook!
Co-authored-by: Ashwin Srinath <[email protected]>
@gpucibot merge |
Description
Update the introductory documentation to take account of current best practices.
In particular, use
compute
judiciously ondask-cuDF
objects, and explain whywe are doing so.
Closes #4665
Closes #4946
Closes #10614
Closes #11931
Checklist