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.

Sign up for GitHub

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

Jump to bottom

add cuXfilter to PyPI packages #412

Closed
wants to merge 3 commits into from
Closed

add cuXfilter to PyPI packages #412

wants to merge 3 commits into from

Conversation

AjayThorve
Copy link
Member

This PR adds cuXfilter to the list of PyPI packages, since it builds wheels now.

I'll keep this open as a draft until the stable version of cuXfilter is released on pypi.nvidia.com

@jarmak-nv
Copy link
Contributor

I can't review unchanged lines, but can you also add cuxfilter to the pip note on line 639
https://github.com/rapidsai/docs/pull/412/files#diff-395138901913dbcb12c6504c460d16462b17ad056b0ddaca459af85680d250beR639

@AjayThorve
Copy link
Member Author

I can't review unchanged lines, but can you also add cuxfilter to the pip note on line 639 https://github.com/rapidsai/docs/pull/412/files#diff-395138901913dbcb12c6504c460d16462b17ad056b0ddaca459af85680d250beR639

@jarmak-nv done

@jarmak-nv
Copy link
Contributor

jarmak-nv commented Jul 12, 2023 •
edited
Loading

And on the install page we should change from:

RAPIDS 23.06 pip packages of cuDF, dask-cuDF, cuML, cuGraph, cuSpatial, RMM, and RAFT are available for CUDA 11 and CUDA 12 on the NVIDIA Python Package Index.

to:

RAPIDS pip packages are available for CUDA 11 and CUDA 12 on the NVIDIA Python Package Index.

https://deploy-preview-412--docs-rapids-ai.netlify.app/install#pip

@AjayThorve
Copy link
Member Author

RAPIDS pip packages are available for CUDA 11 and CUDA 12 on the NVIDIA Python Package Index

Done, thanks!

@jarmak-nv
Copy link
Contributor

@AjayThorve any objections if I merge these changes into #414? Then we can close this and simplify the 23.08 release merging.

@AjayThorve
Copy link
Member Author

@AjayThorve any objections if I merge these changes into #414? Then we can close this and simplify the 23.08 release merging.

@jarmak-nv Nope, it's good to merge. Thanks!

@AjayThorve AjayThorve marked this pull request as ready for review July 27, 2023 16:36
@AjayThorve AjayThorve requested a review from a team as a code owner July 27, 2023 16:36
jarmak-nv added a commit to jarmak-nv/docs that referenced this pull request Jul 27, 2023
@jarmak-nv
Merge PR rapidsai#412
76b7113
@jarmak-nv
Copy link
Contributor

Done! You can now close this, ty for the work on it!

@AjayThorve AjayThorve closed this Jul 27, 2023
@jarmak-nv jarmak-nv mentioned this pull request Jul 27, 2023
Merged
4 tasks
raydouglass added a commit that referenced this pull request Aug 14, 2023
@jarmak-nv @bdice
@raydouglass @jakirkham @ajschmidt8
23.08 Release Selector and Install Page Updates (#414)
509f416 
closes #413 

This PR contains the updates to the release selector required for the
conda-forge CUDA 12.0 work as well as the docker overhaul.

To do:
- [x] CUDA 12 conda-forge updates
- [x] Docker Overhaul updates to the selector
- [x] Docker overhaul updates to the install page
- [x] `23.08` release updates

This PR updates:
- Selector
   - Enables CUDA 12.0 for conda packages
   - Disables CUDA 12.0 for arm conda packages
   - Disables cuSignal from conda CUDA 12.0
   - Removes the note on cuSignal for conda CUDA 12.0
- Removes `PyCaret` as a conda additional package, it's now only
available on pip for the newest versions
      - https://pypi.org/project/pycaret/#history
      - https://anaconda.org/conda-forge/pycaret/files
   - Updated to new docker setup
      - Much simpler, only `base` and `notebooks` 
      - No OS selection
      - Same CUDA 12 restrictions as Conda
 - Install page
    - Adds 2 troubleshooting steps about conda CUDA 12
       - No ARM
       - PyTorch CUDA version compatibilities
- Expands the CUDA version compatibility section and breaks it up into
docker/conda and pip

I've also merged in #412 to simplify merging at release.

---------

Co-authored-by: Bradley Dice <bdice@bradleydice.com>
Co-authored-by: Ray Douglass <3107146+raydouglass@users.noreply.github.com>
Co-authored-by: jakirkham <jakirkham@gmail.com>
Co-authored-by: AJ Schmidt <aschmidt@nvidia.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@AjayThorve @jarmak-nv