-
Notifications
You must be signed in to change notification settings - Fork 9
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
1232 Implement Storable collection trait in support of CUDA streams #1236
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #1236 +/- ##
===========================================
+ Coverage 81.11% 81.25% +0.14%
===========================================
Files 733 738 +5
Lines 28144 28211 +67
===========================================
+ Hits 22828 22923 +95
+ Misses 5316 5288 -28
|
PR tests (nvidia cuda 11.0, ubuntu, mpich)
|
PR tests (nvidia cuda 10.1, ubuntu, mpich)
|
PR tests (intel 19, ubuntu, mpich)
|
PR tests (intel 18.03, ubuntu, mpich)
|
PR tests (nvidia cuda 10.1, ubuntu, mpich)
|
PR tests (intel 18.03, ubuntu, mpich)
|
PR tests (intel 19, ubuntu, mpich)
|
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.
looks good overall 👍 please have a look at the questions posted
If we wanted to not use string-based keys, we could use tag types with a registry instance. |
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.
I'll be happy to have this for the CUDA support work. Looks great.
0cd8cc6
to
688890b
Compare
PR tests (nvidia cuda 11.0, ubuntu, mpich)
|
PR tests (intel 18.03, ubuntu, mpich)
|
PR tests (nvidia cuda 10.1, ubuntu, mpich)
|
PR tests (intel 19, ubuntu, mpich)
|
Fixes #1232
Implement a general mechanism to store data with a collection element. Implemented with a key/value store that detects whether the value needs serialization or not. The string-based key could add some overhead, but I think it's fine for the use cases we have.