Open chrisklus opened 5 years ago
Our plan is to do this after the code review. I'll leave it unassigned for now, and will assign to @chrisklus when we're getting close on this.
Please see https://docs.google.com/document/d/1OfHhIDuaBt-hhXVtpnJgM3Wm0iByJ7dmXd-2UkphUR4/edit# for a reference of Graphing Quadratics PhET-iO design.
And see https://github.com/phetsims/phet-io/blob/master/doc/how-to-design-phet-io-features-for-a-simulation.md for instructions for how to design for PhET-iO.
A few weeks ago we discussed whether this sim should go to RC with both PhET and PhET-iO brands. At the time, we decided to defer that decision until we were closer to publication. @chrisklus said that the instrumentation of the sim itself would not block the PhET-iO brand testing and publication, so it is the state of the common code and PhET-iO library code that would likely be the determining factor.
In the 5/16/2019 developer meeting, I said that I think this sim is about two weeks away from an RC, and asked if the PhET-iO code infrastructure was likely to be in a state where PhET-iO brand could and should be published. @pixelzoom pointed out that Graphing Quadratics is currently blocked from publishing PhET-iO brand due to several issues, including:
My impression from this discussion was that it is pretty unlikely that things will be in a state to support the simultaneous testing and release of the PhET and PhET-iO brands. My recommendation would be to plan to only go for PhET brand for the first release.
Assigning to @ariel-phet to see if he concurs. Tagging @arnabp and @kathy-phet so that they are aware and can comment if they feel the need.
@jbphet @arnabp I concur. PhET-brand should be published without worrying about publishing an iO brand simultaneously.
The sim was published without iO. Assigning to @chrisklus to take notice when the noted PhET-iO issues have been resolved to a state so the PhET-iO brand of blackbody can be published.
Notes for 9/27/2018 meeting with @kathy-phet, @ariel-phet< @DianaTavares, @samreid, @chrisklus, @arouinfar, and myself.
The following items should be instrumented:
We discussed whether the values that are shown on the X and Y axes should be instrumented, thus allowing them to be hidden. We decided against doing this, but would be willing to do it if a client requested this feature.
We also discussed whether to allow hiding of the curve, and decided against, but would add it if a client requested it.