-
Notifications
You must be signed in to change notification settings - Fork 135
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
Widen histogram agent constructor to more types #2380
Conversation
This allows to accept more data types beyond arrays of exact static sizes. No SASS changes on CUB device histogram test with CTK 12.6. Fixes NVIDIA#1877 for AgentHistogram
🟨 CI finished in 3h 52m: Pass: 99%/251 | Total: 1d 16h | Avg: 9m 42s | Max: 47m 51s | Hits: 98%/24387
|
Project | |
---|---|
CCCL Infrastructure | |
libcu++ | |
+/- | CUB |
Thrust | |
CUDA Experimental | |
pycuda | |
CUDA C Core Library |
Modifications in project or dependencies?
Project | |
---|---|
CCCL Infrastructure | |
libcu++ | |
+/- | CUB |
+/- | Thrust |
CUDA Experimental | |
+/- | pycuda |
+/- | CUDA C Core Library |
🏃 Runner counts (total jobs: 251)
# | Runner |
---|---|
178 | linux-amd64-cpu16 |
42 | linux-amd64-gpu-v100-latest-1 |
16 | linux-arm64-cpu16 |
15 | windows-amd64-cpu16 |
🟩 CI finished in 6h 06m: Pass: 100%/251 | Total: 1d 16h | Avg: 9m 39s | Max: 47m 51s | Hits: 98%/24387
|
Project | |
---|---|
CCCL Infrastructure | |
libcu++ | |
+/- | CUB |
Thrust | |
CUDA Experimental | |
pycuda | |
CUDA C Core Library |
Modifications in project or dependencies?
Project | |
---|---|
CCCL Infrastructure | |
libcu++ | |
+/- | CUB |
+/- | Thrust |
CUDA Experimental | |
+/- | pycuda |
+/- | CUDA C Core Library |
🏃 Runner counts (total jobs: 251)
# | Runner |
---|---|
178 | linux-amd64-cpu16 |
42 | linux-amd64-gpu-v100-latest-1 |
16 | linux-arm64-cpu16 |
15 | windows-amd64-cpu16 |
just to make sure I'm not missing something. The idea here is to extend the constructor to anything that can be converted to a pointer. |
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.
LGTM
Yes. Previously, the constructor only accepted a C-style array of matching length. Now the constructor accepts a contiguous sequence of data from all kinds of sources. This allows us to use the |
This allows to accept more data types beyond arrays of exact static sizes.
No SASS changes on CUB device histogram test with CTK 12.6.
Fixes #1877 for AgentHistogram