From 48c9c4c82205e628fa9fb9d754dfa0afcd2a3be6 Mon Sep 17 00:00:00 2001 From: pixelzoom Date: Mon, 20 Sep 2021 17:00:00 -0600 Subject: [PATCH] add missing default options and merge, https://github.com/phetsims/fourier-making-waves/issues/165 --- js/wavepacket/model/WavePacket.js | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/js/wavepacket/model/WavePacket.js b/js/wavepacket/model/WavePacket.js index 4a624d29..1608d886 100644 --- a/js/wavepacket/model/WavePacket.js +++ b/js/wavepacket/model/WavePacket.js @@ -16,6 +16,8 @@ import DerivedProperty from '../../../../axon/js/DerivedProperty.js'; import NumberProperty from '../../../../axon/js/NumberProperty.js'; import Range from '../../../../dot/js/Range.js'; +import merge from '../../../../phet-core/js/merge.js'; +import Tandem from '../../../../tandem/js/Tandem.js'; import ArrayIO from '../../../../tandem/js/types/ArrayIO.js'; import NumberIO from '../../../../tandem/js/types/NumberIO.js'; import fourierMakingWaves from '../../fourierMakingWaves.js'; @@ -35,7 +37,9 @@ class WavePacket { */ constructor( options ) { - //REVIEW: options.tandem looks required, acceptable to do a merge with an opt-out tandem here? + options = merge( { + tandem: Tandem.REQUIRED + }, options ); // @public this.L = 1; // wavelength when component spacing is 2 * Math.PI, in m