Skip to content
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

Trick prints warning for checkpoints outside of freeze mode. #1682

Open
wants to merge 13 commits into
base: master
Choose a base branch
from

Conversation

Mrockwell2
Copy link
Contributor

No description provided.

@coveralls
Copy link

coveralls commented Mar 29, 2024

Coverage Status

coverage: 56.404% (+0.5%) from 55.945%
when pulling e77bb07 on Mrockwell2:1461-only-allow-dumping-and-loading-checkpoints-in-freeze-mode
into c4e60d9 on nasa:master.

@excaliburtb
Copy link
Contributor

i don't think this is true for generic cases. if you have some fully async threads running, then it might be a problem. otherwise, it is well-defined behavior. Perhaps you have a specific use case or example that illustrates the issue?

@Mrockwell2
Copy link
Contributor Author

i don't think this is true for generic cases. if you have some fully async threads running, then it might be a problem. otherwise, it is well-defined behavior. Perhaps you have a specific use case or example that illustrates the issue?

When you checkpoint a running sim without freezing and reload that checkpoint, the data recording log resumes on the next frame. So far that's the most glaring thing I've found, but I'll keep looking.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants