Closed jbphet closed 7 years ago
@amanda-phet it's my understanding that level 2 is not yet supposed to have left over terms in the carousel. Only level 8 is specified to not use all the available terms. If that's true the last challenge on level 2 has one too many "y" terms.
The targets add to 4 "y" terms, but there are five in the carousel. Maybe it should be "2{y}, 1{2y}"?
@amanda-phet Same deal in the last challenge of level 4. There are 4 "y2" terms in the targets, but 5 in the carousel.
@amanda-phet Level 5, I also could not think of a way to not have left overs in the first challenge. I always had 2{1} left over.
@jbphet, third challenge of level five, first target should be "y-2x", not "y-2y".
@amanda-phet first challenge of level six, I always had some combination leading to -2x being left over (whether it was 1{-2x} or 2{-x})
@jbphet 3rd challenge of level 6, carousel should contain one "-4z", not "-3z".
@amanda-phet the 4th challenge of level 6 is an interesting one. The left over terms equate to 0 (1{y}, 1{-y}). I can go out of my way to use them, but the simplest solution leaves them unused. This one may be okay to not change, but still noting it.
@amanda-phet the last challenge in level 6 is my first unsolvable prompt. I do not have the 12 "y" terms the targets are asking for, I only have 11.
@jbphet 3rd challenge of level 7 wants 2{3}, not 1{3}.
@amanda-phet the 4th challenge of level 8 is unsolvable. I cannot make a positive "2" term for the first target.
@jbphet I've finished and problems are noted in the string of comments above. I'll edit the first post to include a checklist of comments to make this easier to read.
Thanks @phet-steele - that was some very thorough testing.
I've addressed the three data-entry issues that were noted, and will unassign myself until @amanda-phet has addressed the other issues. At that point, she should assign back to me and I'll implement the changes.
Thanks @phet-steele . I wasn't sure how to address the issues here in github but I made changes in the design doc and highlighted them. Assigning to @jbphet to review the changes in the design doc and update the code.
The code has now been updated based on the updates/corrections in the design document. Assigning back to @phet-steele to run through these again and see if they are all now correct.
Everything is correct! Thanks @jbphet and @amanda-phet.
Thanks @phet-steele.
The game screen has 8 levels with 5 challenges each for a total of 40 challenges. Each challenge has 3 equations that the user needs to construct to complete the challenge, so there are 120 equations. I've entered all of these into the code, but I did it quickly, and probably didn't get them all right. I could use some help verifying them.
I've published a version where the order of the challenges is not randomized, so they should match the order in which they are described in the spec, which should make them easier to check. The spec can be found at https://docs.google.com/document/d/1ppP9pgm0Nc0SEZbPZnrKhhXZ9JKKzurfxg7_Dvbh_rM/edit#, and the challenges are described in the tables in the "Level Overview" section. Please verify that challenges that are described in this document match those that appear in version http://www.colorado.edu/physics/phet/dev/html/expression-exchange/1.0.0-dev.19/expression-exchange_en.html of the sim. The equations and the carousel contents should be verified.
Note that the game cannot be played yet - the collection areas are not yet operational.
EDIT read this to navigate this easier @jbphet and @amanda-phet:
For @jbphet:
For @amanda-phet: