-
Notifications
You must be signed in to change notification settings - Fork 6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Instrument for PhET-iO #50
Comments
We just want to release this for sonification work, not necessarily testing it completely and fully. On hold until we figure out https://github.com/phetsims/phet-io/issues/1170 |
We decided to go ahead and ready a phet-io dev version that can be used with the sonification process. I will talk with @phet-steele about what a phet-io dev test would look like. In the mean time I'll make a dev test. |
http://www.colorado.edu/physics/phet/dev/html/ohms-law/1.4.0-phetiodev.1/wrappers/index/ tested in phetsims/qa#18 on hold until that gets sorted out |
http://www.colorado.edu/physics/phet/dev/html/ohms-law/1.4.0-phetiodev.3/wrappers/index/ has been released. This is not a published version, but a dev version. I will mark it in the spreadsheet as fully instrumented, even though there is not yet a deliverable link on the phet-io website. Closing. |
@emily-phet and I discussed and we think that this would be a good step to take to advance the sonification side of accessibility on this sim.
https://github.com/phetsims/phet-io/blob/ee0a5b85ea06039c2871a8d3d95207c1dfc73e33/doc/how-to-instrument-a-phet-simulation-for-phet-io.md
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
insteadchipper/data/test-phetio
grunt --brand=phet-io
and test the built version by launching build/wrappers/index and testing all the linksThe text was updated successfully, but these errors were encountered: