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.

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

[DOC] 10 Minutes to cuDF and Dask-cuDF out of date #10527

Closed
sophwats opened this issue Mar 28, 2022 · 2 comments
Closed

[DOC] 10 Minutes to cuDF and Dask-cuDF out of date #10527

sophwats opened this issue Mar 28, 2022 · 2 comments
Assignees
Labels
bug Something isn't working doc Documentation Python Affects Python cuDF API.

Comments

@sophwats
Copy link

sophwats commented Mar 28, 2022

10 Minutes to cuDF And Dask-cuDF notebook returns an error when executed

Location of incorrect documentation
Notebook is here in docs
and here in git
Describe the problems or issues found in the documentation

  1. cell 53 gives a recursion error: RecursionError: maximum recursion depth exceeded while calling a Python object
  2. cell 65 and 66 give an attribute error: AttributeError: DataFrame object has no attribute as_matrix

Steps taken to verify documentation is incorrect
cloned the cuDF repo, executed the notebook using Restart Kernel and Run All Cells from the Kernel menu.
executing in a 22.04 RAPIDS kernel, built using RAPIDS Release selector tool, with conda and 22.04 (Nightly) on 8th March 2022.

Suggested fix for documentation

to fix 2. above, the .as_matrix() in cells 65 and 66 should be replaced by .as_numpy()

@sophwats sophwats added Needs Triage Need team to review and classify doc Documentation labels Mar 28, 2022
@shwina shwina self-assigned this Mar 29, 2022
@shwina shwina added bug Something isn't working Python Affects Python cuDF API. and removed Needs Triage Need team to review and classify labels Mar 29, 2022
@shwina
Copy link
Contributor

shwina commented Mar 29, 2022

@sophwats Thanks for reporting. I addressed (2) in PR #10531, but I couldn't reproduce (1) using the latest build of cuDF 22.04. Perhaps it was fixed in the interim?

@sophwats
Copy link
Author

Agreed - I couldn't reproduce (1) on nightly this morning. Thanks for fixing (2).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working doc Documentation Python Affects Python cuDF API.
Projects
None yet
Development

No branches or pull requests

2 participants