Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Checkpoint/Resume #2081
Checkpoint/Resume #2081
Changes from all commits
1145f0c
25005b7
6d3030f
5a9fd9b
1e5c3e3
60c918d
99bd50f
837c1fe
304cf48
90b882c
3794cd9
97aa29d
930e90a
ab6100e
61bcbb6
97c5b92
adecbff
f4e0c2b
a31d318
7385dc7
43b1538
e5257f9
600c398
fe90ae1
eb7b854
5a403df
70443fd
5aec9f4
75b3a61
b611178
3ccd4d2
48527f8
feda5eb
7275eaf
6a1fb50
9b7353a
05e7423
5ed83e7
2f74882
2754ac2
48850c9
d6e0376
88450cf
a5c766e
c41b8aa
8676056
524d581
1b44eff
c972bd0
c3ed29e
1a45b8c
324cbe9
782f66e
2c7b7b1
7ee0d7c
6d44618
a36851b
19ef79e
1cd59b0
36e3762
b5a47b4
8b7c9c8
4f90020
125a90a
99b1883
06849da
d090082
b23487c
9313422
6c49379
3b1be79
1ca974a
31617e9
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
Wouldn't it be less error-prone if the base class template
event_stream_base
had its own serialization hooks which could be called from here?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.
Hmm. It's a tradeoff. In the current way I can use the
ENABLE
macro in themulticore
subclassand need a custom serializer in the
gpu
case. Your proposal is slightly more code to write:ENABLE
in basemulticore
gpu
minus the base members plus one line for serializingbase
The problem arises when writing out the event spans. One is a GPU memory view aka
(ptr, length)
and the multicore is a range aka
(ptr_beg, ptr_end)
. It seems prudent to maybe merge representationsfirst?
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.
yeah, I see your point. Let's leave it the way it is.