From 54621eecf1002c75418c2f12a52644ba4ac7c705 Mon Sep 17 00:00:00 2001
From: Sam Reid <reids@colorado.edu>
Date: Wed, 6 Mar 2024 07:29:57 -0700
Subject: [PATCH] Change DynamicProperty declarations to less restrictive
 types, see https://github.com/phetsims/projectile-data-lab/issues/222

---
 js/common-vsm/model/VSMModel.ts          | 26 ++++++++++++++++--------
 js/common-vsm/view/CustomLauncherNode.ts |  4 ++--
 js/common/view/LauncherNode.ts           |  2 +-
 3 files changed, 20 insertions(+), 12 deletions(-)

diff --git a/js/common-vsm/model/VSMModel.ts b/js/common-vsm/model/VSMModel.ts
index 30373ff2..79ad92b7 100644
--- a/js/common-vsm/model/VSMModel.ts
+++ b/js/common-vsm/model/VSMModel.ts
@@ -31,6 +31,8 @@ import stopwatchTick_mp3 from '../../../sounds/stopwatchTick_mp3.js';
 import soundManager from '../../../../tambo/js/soundManager.js';
 import PDLQueryParameters from '../../common/PDLQueryParameters.js';
 import PDLPreferences from '../../common/PDLPreferences.js';
+import TReadOnlyProperty from '../../../../axon/js/TReadOnlyProperty.js';
+import PhetioProperty from '../../../../axon/js/PhetioProperty.js';
 
 const tickSound = new SoundClip( stopwatchTick_mp3, { initialOutputLevel: 1 } );
 soundManager.addSoundGenerator( tickSound );
@@ -56,19 +58,25 @@ export default class VSMModel<T extends VSMField> extends PDLModel<T> {
   public readonly measuringTapeBasePositionProperty;
   public readonly measuringTapeTipPositionProperty;
 
-  public readonly latestLaunchAngleProperty: DynamicProperty<number, number, T>;
-  public readonly latestLaunchSpeedProperty: DynamicProperty<number, number, T>;
+  ////////////////////////////////////////////////////////////////////////////////////////
+  // These DynamicProperties select based on the selected VSMField
+
+  public readonly latestLaunchAngleProperty: TReadOnlyProperty<number>;
+  public readonly latestLaunchSpeedProperty: TReadOnlyProperty<number>;
 
   // The angleStabilizerProperty represents the amount of angle stabilization applied to the launcher. The value is between 0 and 1, where 0 means minimum stabilization and 1 means maximum stabilization.
-  public readonly angleStabilizerProperty: DynamicProperty<number, number, T>;
-  public readonly standardDeviationAngleProperty: DynamicProperty<number, number, T>;
+  public readonly angleStabilizerProperty: PhetioProperty<number>;
+  public readonly standardDeviationAngleProperty: TReadOnlyProperty<number>;
+
+  public readonly selectedProjectileNumberProperty: PhetioProperty<number>;
+  public readonly selectedProjectileProperty: TReadOnlyProperty<Projectile | null>;
+  public readonly numberOfLandedProjectilesProperty: TReadOnlyProperty<number>;
+  public readonly totalProjectileCountProperty: TReadOnlyProperty<number>;
 
-  public readonly selectedProjectileNumberProperty: DynamicProperty<number, number, T>;
-  public readonly selectedProjectileProperty: DynamicProperty<Projectile | null, Projectile | null, T>;
-  public readonly numberOfLandedProjectilesProperty: DynamicProperty<number, number, T>;
-  public readonly totalProjectileCountProperty: DynamicProperty<number, number, T>;
+  public readonly launcherProperty: PhetioProperty<Launcher>;
 
-  public readonly launcherProperty: DynamicProperty<Launcher, Launcher, T>;
+  // End of DynamicProperties
+  ////////////////////////////////////////////////////////////////////////////////////////
 
   //REVIEW constructor should be protected
   public constructor( fields: T[], providedOptions: VSMModelOptions<T> ) {
diff --git a/js/common-vsm/view/CustomLauncherNode.ts b/js/common-vsm/view/CustomLauncherNode.ts
index 4fe892b6..5740f3fa 100644
--- a/js/common-vsm/view/CustomLauncherNode.ts
+++ b/js/common-vsm/view/CustomLauncherNode.ts
@@ -54,8 +54,8 @@ export default class CustomLauncherNode extends LauncherNode {
                       mysteryOrCustomProperty: TProperty<MysteryOrCustom>,
                       mysteryLauncherProperty: TProperty<Launcher>,
                       launcherMechanismProperty: TProperty<LauncherMechanism>,
-                      standardDeviationAngleProperty: TProperty<number>,
-                      latestLaunchSpeedProperty: TProperty<number>,
+                      standardDeviationAngleProperty: TReadOnlyProperty<number>,
+                      latestLaunchSpeedProperty: TReadOnlyProperty<number>,
                       fieldProperty: TReadOnlyProperty<Field> | null,
                       providedOptions?: CustomLauncherNodeOptions ) {
 
diff --git a/js/common/view/LauncherNode.ts b/js/common/view/LauncherNode.ts
index 5bda3d87..cee3d45d 100644
--- a/js/common/view/LauncherNode.ts
+++ b/js/common/view/LauncherNode.ts
@@ -77,7 +77,7 @@ export default class LauncherNode extends Node {
   public constructor( private readonly modelViewTransform: ModelViewTransform2,
                       private readonly meanLaunchAngleProperty: TProperty<number>,
                       launcherHeightProperty: TProperty<number>,
-                      mysteryLauncher: TProperty<Launcher>,
+                      mysteryLauncher: TReadOnlyProperty<Launcher>,
                       fieldProperty: TReadOnlyProperty<Field> | null,
                       providedOptions?: LauncherNodeOptions ) {