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

Only allow dumping and loading checkpoints in freeze mode #1461

Open
jdeans289 opened this issue Feb 22, 2023 · 0 comments · May be fixed by #1727
Open

Only allow dumping and loading checkpoints in freeze mode #1461

jdeans289 opened this issue Feb 22, 2023 · 0 comments · May be fixed by #1727
Assignees

Comments

@jdeans289
Copy link
Contributor

If you dump or reload a checkpoint while not in freeze mode, you can have unexpected results. Currently, the sim control gui enforces this by greying out buttons, but there is no mechanism preventing a checkpoint dump or reload in run mode in Trick itself, so manually calling dump_checkpoint in run mode can result in confusing bugs.

This issue can be closed when:

  • Trick prints a warning and refuses to dump/load a checkpoint if not in freeze mode
  • This behavior is tested and documented
@Mrockwell2 Mrockwell2 self-assigned this Mar 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants