-
Notifications
You must be signed in to change notification settings - Fork 301
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
force atlas notebook changes to run in cugraph 23.08 container. #3656
Conversation
…ightly container.
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
You need either a note that this notebook only running in the rapids docker container or provide a list of how to install the needed packages |
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.
update install notes
force atlas notebook changes to run in cugraph 23.08 containerWhich container (RAPIDS runtime, DLFW, etc.)? Can that be mentioned somewhere? |
I tried to review but GH gave this message: I'm curious why the diff was so large and how many changes were required to get this NB to run in the container environment you're using. Can you maybe update this PR description to summarize the changes? (looks like the description needs updating anyway since it mentions filing an issue, |
/merge |
changes to allow for libary dependence on no longer supported numpy functions.
Also addresses notebook's need for cuxfilter.
resolves #3653