phetsims / circuit-construction-kit-common

"Circuit Construction Kit: Basics" is an educational simulation in HTML5, by PhET Interactive Simulations.
GNU General Public License v3.0
10 stars 10 forks source link

Fuse blows repeatedly in State wrapper #849

Closed Nancy-Salpepi closed 1 year ago

Nancy-Salpepi commented 2 years ago

Test device iPad Air 2

Operating System 15.2

Browser Safari

Problem description For https://github.com/phetsims/qa/issues/772 In the State Wrapper, the fuse will continuously blow in the downstream sim. I was able to reproduce this using my MacBook + Safari. When I looked into this in the Studio wrapper, if I blew the fuse before launching the sim (not sure why anyone would want to do that), the sim launches showing the fuse being blown.

Steps to reproduce

  1. In the State Wrapper: Make a circuit with a fuse.
  2. Increase the battery voltage until the fuse blows (or decrease the current rating)

Visuals fuseblows

KatieWoe commented 2 years ago

If the fuse isn't connected, and should be blown, it does not blow in the launched sim.

brokennotconnected
arouinfar commented 2 years ago

@samreid this looks like a bug that can negatively impact the client's usage. I recommend we try fixing this before sending the client another version, but if that's not feasible it should be documented as a known bug.

samreid commented 2 years ago

Good find, QA team! The commit addresses https://github.com/phetsims/circuit-construction-kit-common/issues/849#issuecomment-1032811084.

For https://github.com/phetsims/circuit-construction-kit-common/issues/849#issue-1126577466, my understanding is that the Fuse is a dynamic element, and created fresh each frame. It is created with isTrippedProperty=false by default. Then when the state sets its value to isTripped=true that triggers the animation. This may not be a problem for clients in practice, since there is no use case for clients using the state wrapper with students. In studio this bug would present as the fuse just tripping when the customized sim starts up (though I did notice that animation is at the wrong coordinates). But maybe we can say "only play the animation if the fuse trips when we aren't setting phet-io state". @arouinfar should we try that, or do we sometimes need to show the fuse trip animation when setting phet-io state? Please advise.

arouinfar commented 2 years ago

Thanks for the explanation @samreid. I wasn't sure if this bug would also manifest if a student submits the state of the sim as a solution. If this bug is unlikely to show up in that case, I'll update the doc we shared with the client. We can defer the discussion about the tripped fuse animation until we resume work on the next milestone.

samreid commented 2 years ago

Sounds good, I'll move to the next milestone. Please feel free to move it back if you still need to update the doc.

samreid commented 1 year ago

Fixed, and working well in testing launching a studio state with a tripped fuse. Closing.