Closed samreid closed 7 years ago
I have done many steps in the check list. I need to make sure that @samreid and I work on https://github.com/phetsims/phet-io/issues/1080 in this issue.
I emailed the skype channel:
MK & I are preparing Resistance in a Wire for phet & phet-io rc testing. We are trying to decide if we should use the the legacy process (as was used for molarity) which has separate branches and dependencies.json for each brand, or if we should test the waters with using just one branch and one dependencies.json.
@jonathanolson recommended using the legacy process for Resistance in a Wire until we have had more time to consider finalize the "one branch" process.
The rc phet-io version has been deployed and has been sent into the test queue. Closing.
This is a small and simple simulation and would be a great introduction to sim instrumentation for @zepumph. Would be good to collaborate on to start with. Generally, we'll paste the checklist from https://github.com/phetsims/phet-io/blob/36d9b4cb5e01107bd5fab43b0fb1a54b14d73e69/doc/how-to-instrument-a-phet-simulation-for-phet-io.md into this comment and follow those directions.
How to Instrument a PhET Simulation for PhET-iO
Overview of instrumentation steps:
/blob/<SHA>/
so that the specific guide you used is preserved.?phetioValidateTandems=false
since some common code-components don't yet have tandems.dt
values are used instead of Date.now() or other Date functions. This is necessary for reproducible playback via input events.phet.joist.random
, and all doing so after modules are declared (non-statically)? For instance, the following methods (and perhaps others) should not be used:Math.random
,_.shuffle
,_.sample
,_.random
undefined
values cannot be saved by phet-io, sims should be written to usenull
insteadgrunt --brand=phet-io
and test the built version by launching build/wrappers/index and testing all the links