Adding pixel representation to the phetio state, see #42

phetsims · Oct 4, 2024 · d8dc0fb · d8dc0fb
1 parent e15e50c
commit d8dc0fb
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/js/coins/view/CoinExperimentMeasurementArea.ts b/js/coins/view/CoinExperimentMeasurementArea.ts
@@ -206,6 +206,10 @@ export default class CoinExperimentMeasurementArea extends VBox {
       }
     } );
 
+    sceneModel.coinSet.measuredDataChangedEmitter.addListener( () => {
+      this.measuredCoinsPixelRepresentation.redraw( sceneModel.coinSet.measuredValues );
+    } );
+
     sceneModel.preparingExperimentProperty.link( () => this.measuredCoinsPixelRepresentation.abortAllAnimations( 0 ) );
 
     // Create the node that will be used to cover (aka "mask") the coin so that its state can't be seen.

diff --git a/js/coins/view/CoinSetPixelRepresentation.ts b/js/coins/view/CoinSetPixelRepresentation.ts
@@ -39,6 +39,8 @@ export default class CoinSetPixelRepresentation extends CanvasNode {
     const fps = 40;
     const totalFrames = 4 * MEASUREMENT_PREPARATION_TIME / fps;
 
+
+    // TODO: Try isSettingPhetioStateProperty.value ? 0 : MEASUREMENT_PREPARATION_TIME, https://github.com/phetsims/quantum-measurement/issues/39
     this.populatingAnimation = new Animation( {
       to: totalFrames,
       duration: MEASUREMENT_PREPARATION_TIME,