-
Notifications
You must be signed in to change notification settings - Fork 75
Require CUDA 12.2+ #735
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
Require CUDA 12.2+ #735
Conversation
Bump minimum CUDA version from 12.0 to 12.2.
|
|
||
| cuxfilter docker example installation for cuda 12.0, ubuntu 20.04: | ||
| cuxfilter docker example installation for cuda 12, ubuntu 20.04: | ||
|
|
||
| .. code-block:: bash | ||
|
|
||
| docker pull rapidsai/rapidsai:cuda12.0-runtime-ubuntu20.04 | ||
| docker run --gpus all --rm -it -p 8888:8888 -p 8787:8787 -p 8786:8786 \ | ||
| rapidsai/rapidsai:cuda12.0-runtime-ubuntu20.04 | ||
| docker run --gpus all --pull always --rm -it \ | ||
| --shm-size=1g --ulimit memlock=-1 --ulimit stack=67108864 \ | ||
| rapidsai/base:25.10-cuda12-py3.13 | ||
|
|
||
| # open http://localhost:8888 |
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.
These docs are pretty out-of-date relative to our current recommendations
We also let the CUDA minor version float to latest in our containers
Have rewritten this based on our current docs. Though maybe we could just link to our install page instead (to avoid this falling out-of-date again)?
|
Am seeing the following test failure on CI FAILED cuxfilter/tests/test_dashboard.py::TestDashBoard::test_variables - AssertionError: assert {'col_1_chart': '6<=col_1<=9'} == {}
Left contains 1 more item:
{'col_1_chart': '6<=col_1<=9'}
Full diff:
- {}
+ {
+ 'col_1_chart': '6<=col_1<=9',
+ }However the changes here are to the docs and dropping on CI matrix job. So this appears unrelated Could someone please take a closer look? Edit - Appears this is flaky test and is tracked in issue: #592 |
|
/merge |
|
Thanks all! 🙏 |
Bump minimum CUDA version from 12.0 to 12.2.
Part of issue: