Skip to content

Commit

Permalink
Remove unnecessary TODO (covered in #153), see #164
Browse files Browse the repository at this point in the history
  • Loading branch information
samreid committed May 5, 2023
1 parent 3eeba0a commit f419cd3
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions js/variability/model/VariabilityModel.ts
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,6 @@ export default class VariabilityModel extends CAVModel {

public constructor( options: VariabilityModelOptions ) {

// TODO: the parent class sets this incorrectly on reset. See https://github.com/phetsims/center-and-variability/issues/164
// TODO: See https://github.com/phetsims/center-and-variability/issues/117. PhET-iO wants to be able to set these values. Maybe in the preferences, we would also add a "custom"
// option that would allow the user to specify the distribution parameters. Or for PhET-iO, and query parameters
const sceneModels = [
Expand All @@ -50,7 +49,6 @@ export default class VariabilityModel extends CAVModel {

this.variabilitySceneModels = sceneModels;


this.selectedVariabilityProperty = new EnumerationProperty( VariabilityMeasure.RANGE, {
tandem: options.tandem.createTandem( 'selectedVariabilityProperty' )
} );
Expand Down

0 comments on commit f419cd3

Please sign in to comment.