Skip to content

Conversation

@DaceT
Copy link
Contributor

@DaceT DaceT commented Nov 18, 2021

Instructions for installing Dask-SQL w/ RAPIDS so that it can run on GPUs.

Fixes #321

Instructions for installing Dask-SQL w/ RAPIDS so that it can run on GPUs.
@codecov-commenter
Copy link

codecov-commenter commented Nov 18, 2021

Codecov Report

Merging #324 (b5ad937) into main (ec328c6) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #324   +/-   ##
=======================================
  Coverage   95.80%   95.80%           
=======================================
  Files          65       65           
  Lines        2834     2834           
  Branches      426      426           
=======================================
  Hits         2715     2715           
  Misses         74       74           
  Partials       45       45           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ec328c6...b5ad937. Read the comment docs.

Copy link
Collaborator

@charlesbluca charlesbluca left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @DaceT 🙂 looks like there are some linting issues, you should be able to resolve these locally by running pre-commit run --all-files and then pushing the resulting changes up to this branch

With ``GPU Support``
--------------
Create a new conda environment or use an existing one to install RAPIDS with the chosen methods and packages:
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we should maybe add a short blurb about GPU support here, bringing up the following to contextualize the env creation line below:

  • GPU support right now is specifically tied to the RAPIDS libraries
  • It is in development, and generally requires the latest nightlies of cuDF
  • It is experimental, and users should still expect some level of bugs / undefined behavior

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @charlesbluca, I'll see to add them!

Copy link
Collaborator

@charlesbluca charlesbluca left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the work here @DaceT 😄

@charlesbluca charlesbluca merged commit a3fc92d into dask-contrib:main Nov 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[DOC] Describe how to install w/ GPU support

3 participants