-
Notifications
You must be signed in to change notification settings - Fork 190
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
Ensure CuPy arrays can be used with cuda.parallel
too
#2335
Conversation
Co-authored-by: Michael Schellenberger Costa <[email protected]>
Hmmm I would expect most of the CI to not run since only Python files are changed... |
If all of CI runs that is a bug in out filter detection |
I believe I addressed this here #2336
Note that that PR runs all the tests because it changes our core infrastructure |
🟨 CI finished in 4h 59m: Pass: 99%/421 | Total: 2d 06h | Avg: 7m 43s | Max: 1h 08m | Hits: 91%/34420
|
Project | |
---|---|
+/- | CCCL Infrastructure |
libcu++ | |
CUB | |
Thrust | |
CUDA Experimental | |
pycuda |
Modifications in project or dependencies?
Project | |
---|---|
+/- | CCCL Infrastructure |
+/- | libcu++ |
+/- | CUB |
+/- | Thrust |
+/- | CUDA Experimental |
+/- | pycuda |
🏃 Runner counts (total jobs: 421)
# | Runner |
---|---|
304 | linux-amd64-cpu16 |
66 | linux-amd64-gpu-v100-latest-1 |
28 | linux-arm64-cpu16 |
23 | windows-amd64-cpu16 |
🟨 CI finished in 1h 41m: Pass: 99%/421 | Total: 2d 05h | Avg: 7m 39s | Max: 55m 13s | Hits: 91%/34420
|
Project | |
---|---|
+/- | CCCL Infrastructure |
libcu++ | |
CUB | |
Thrust | |
CUDA Experimental | |
pycuda |
Modifications in project or dependencies?
Project | |
---|---|
+/- | CCCL Infrastructure |
+/- | libcu++ |
+/- | CUB |
+/- | Thrust |
+/- | CUDA Experimental |
+/- | pycuda |
🏃 Runner counts (total jobs: 421)
# | Runner |
---|---|
304 | linux-amd64-cpu16 |
66 | linux-amd64-gpu-v100-latest-1 |
28 | linux-arm64-cpu16 |
23 | windows-amd64-cpu16 |
🟩 CI finished in 1d 02h: Pass: 100%/421 | Total: 2d 05h | Avg: 7m 37s | Max: 55m 13s | Hits: 91%/34420
|
Project | |
---|---|
+/- | CCCL Infrastructure |
libcu++ | |
CUB | |
Thrust | |
CUDA Experimental | |
pycuda |
Modifications in project or dependencies?
Project | |
---|---|
+/- | CCCL Infrastructure |
+/- | libcu++ |
+/- | CUB |
+/- | Thrust |
+/- | CUDA Experimental |
+/- | pycuda |
🏃 Runner counts (total jobs: 421)
# | Runner |
---|---|
304 | linux-amd64-cpu16 |
66 | linux-amd64-gpu-v100-latest-1 |
28 | linux-arm64-cpu16 |
23 | windows-amd64-cpu16 |
@gevtushenko I thought it's already merged but just realized it's not. Any concern for merging (once I resolve conflicts)? |
@leofang I also thought it was already merged 😄 No concerns, feel free to merge once CI is green. |
/ok to test |
FYI @miscco I merged with the latest main and yet still seeing a lot of CI pipelines fired up, not sure if this is expected as of today? (I lost track of the filtering progress...) |
This comment was marked as resolved.
This comment was marked as resolved.
nvm it's here https://github.com/NVIDIA/cccl/actions/runs/11377383986/job/31651464063?pr=2335 |
🟩 CI finished in 1h 18m: Pass: 100%/372 | Total: 2d 04h | Avg: 8m 31s | Max: 1h 09m | Hits: 81%/27923
|
Project | |
---|---|
+/- | CCCL Infrastructure |
libcu++ | |
CUB | |
Thrust | |
CUDA Experimental | |
pycuda | |
CCCL C Parallel Library |
Modifications in project or dependencies?
Project | |
---|---|
+/- | CCCL Infrastructure |
+/- | libcu++ |
+/- | CUB |
+/- | Thrust |
+/- | CUDA Experimental |
+/- | pycuda |
+/- | CCCL C Parallel Library |
🏃 Runner counts (total jobs: 372)
# | Runner |
---|---|
298 | linux-amd64-cpu16 |
31 | linux-amd64-gpu-v100-latest-1 |
28 | linux-arm64-cpu16 |
15 | windows-amd64-cpu16 |
* ensure cupy arrays can be used with cuda.parallel too * copy the same comment to all places wherever applicable Co-authored-by: Michael Schellenberger Costa <[email protected]> * ensure all needed imports are shown in the example * add CuPy (+CUDA 12.x) as a test dependency --------- Co-authored-by: Michael Schellenberger Costa <[email protected]>
* ensure cupy arrays can be used with cuda.parallel too * copy the same comment to all places wherever applicable Co-authored-by: Michael Schellenberger Costa <[email protected]> * ensure all needed imports are shown in the example * add CuPy (+CUDA 12.x) as a test dependency --------- Co-authored-by: Michael Schellenberger Costa <[email protected]>
Description
closes #2307
Maybe blocked by #2280?Checklist