-
Notifications
You must be signed in to change notification settings - Fork 217
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
Radiation plugin: Fix checkpointing #4401
Radiation plugin: Fix checkpointing #4401
Conversation
There seems to have been a crash in the CI with some clang compiler - I restarted it. |
@franzpoeschel Are checkpoints with your fix always written as one file per checkpoint? This is a strong requirement for picongpu. Standard IO can be optionally in one file with a time series. |
@psychocoderHPC yes, they are. With this fix, I don't even keep the Series open for checkpointing. |
* No need to keep the Series open for checkpointing, so | ||
* just quickly open it here. | ||
*/ | ||
std::optional<::openPMD::Series> tmp; |
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.
See here @psychocoderHPC
This is the Series for checkpointing, just quickly opened here and then closed again
It's good that you asked René. The parameter |
Alright, these are now the new parameters:
No --restartConfig parameter for now, since noone will use it anyway. |
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.
Tested in - all files are created
Follow-up to #4387
Fix #4397