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

Console errors when pressing Reset All after loading saved state #311

Closed
Nancy-Salpepi opened this issue Jul 7, 2022 · 5 comments
Closed
Assignees

Comments

@Nancy-Salpepi
Copy link

Test device
MacBook Air (m1 chip)

Operating System
macOS 12.4

Browser
safari

Problem description
For phetsims/qa#818, after loading a saved state (from version 1.5 or version 1.4), pressing the Reset All button results in console errors. After a quick check, the sim seems to be working correctly after the errors appear.

Steps to reproduce

  1. In Studio version 1.5.0-dev.1, select the Intro screen.
  2. Open the console
  3. Save the state (Feel free to make changes to the sim, but not necessary)
  4. Load the saved state into Studio version 1.5.0-dev.1
  5. Press the Reset All button

Visuals
Screen Shot 2022-07-07 at 1 08 09 PM

@pixelzoom
Copy link
Contributor

pixelzoom commented Jul 7, 2022

As with #310... This is (presumably) related to migration, a focus of this release. Assigning to @zepumph and @samreid.

@pixelzoom pixelzoom assigned samreid and zepumph and unassigned pixelzoom Jul 7, 2022
@zepumph
Copy link
Member

zepumph commented Jul 7, 2022

This is not specific to migration, you can also reproduce this by loading in a saved file from the same version. It is most likely a regression from phetsims/tandem#264. I'll take a look.

@zepumph
Copy link
Member

zepumph commented Jul 7, 2022

Fixed above, since we were now doing disposal, we have to be less sloppy about our validation and error checking. @Nancy-Salpepi you should be able to confirm this is fixed on master, you can load the in the same old version standard phet-io wrapper (output from studio save button) as before. Feel free to close.

zepumph added a commit to phetsims/tandem that referenced this issue Jul 7, 2022
@zepumph zepumph assigned Nancy-Salpepi and unassigned samreid and zepumph Jul 7, 2022
@Nancy-Salpepi
Copy link
Author

Looks fixed @zepumph!
Closing

@pixelzoom
Copy link
Contributor

@Nancy-Salpepi Sounds like you meant to close this issue, so I'll do so.

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

No branches or pull requests

4 participants