-
Notifications
You must be signed in to change notification settings - Fork 76
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
[TESTING] PR to test and work out new CUDA versions for CI #60
[TESTING] PR to test and work out new CUDA versions for CI #60
Conversation
@mike-wendt questions about the new setup -
|
Yes the conda environment has numpy already.
Are you sure? Can you point to the logs?
Triggering the CUDA 9.0 & 10.1 is manual right now. We want to iron out the builds before enabling it globally. |
If you look at the CMake logs, it says using CUDA 10.1
Perfect, just wanted to make sure that out other merges won't be blocked till these issues are ironed out. |
e22e710
to
db80e17
Compare
@raydouglass @mike-wendt made changes to support CUDA 9.0. can you re-trigger this PR with the different CUDA 9.0 and CUDA 10.1 jobs? I realized that what we'll actually need is CUDA 9.0, CUDA 9.2 and CUDA 10.1 ideally |
- Disable logging module for CUDA files since there's a spdlog header incompatibility with nvcc in CUDA < 9.2.
db80e17
to
43836f3
Compare
rerun tests |
2 similar comments
rerun tests |
rerun tests |
New set of tests are passing. Failing tests are old status checks that can't be removed. |
No description provided.