Skip to content
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.

Sign up for GitHub

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

Jump to bottom

CT value failed isValidValue: 11 #5

Closed
KatieWoe opened this issue Oct 31, 2019 · 1 comment
Closed

CT value failed isValidValue: 11 #5

KatieWoe opened this issue Oct 31, 2019 · 1 comment
Assignees
@chrisklus
Labels
type:automated-testing

Comments

@KatieWoe
Copy link 

number-play : fuzz : require.js : run
Query: brand=phet&ea&fuzz&memoryLimit=1000
Uncaught Error: Assertion failed: value failed isValidValue: 11
Error: Assertion failed: value failed isValidValue: 11
    at window.assertions.assertFunction (https://bayes.colorado.edu/continuous-testing/snapshot-1572495798551/assert/js/assert.js?bust=1572509331188:22:13)
    at Object.isValueValid (https://bayes.colorado.edu/continuous-testing/snapshot-1572495798551/axon/js/ValidatorDef.js?bust=1572509331254:300:41)
    at validate (https://bayes.colorado.edu/continuous-testing/snapshot-1572495798551/axon/js/validate.js?bust=1572509331254:30:20)
    at NumberProperty.assertNumberPropertyValidateValue (https://bayes.colorado.edu/continuous-testing/snapshot-1572495798551/axon/js/NumberProperty.js?bust=1572509331254:76:26)
    at NumberProperty.set (https://bayes.colorado.edu/continuous-testing/snapshot-1572495798551/axon/js/NumberProperty.js?bust=1572509331254:98:54)
    at NumberProperty.set value [as value] (https://bayes.colorado.edu/continuous-testing/snapshot-1572495798551/axon/js/Property.js?bust=1572509331254:335:34)
    at OnesPlayArea.calculateTotal (https://bayes.colorado.edu/continuous-testing/snapshot-1572495798551/number-play/js/common/model/OnesPlayArea.js?bust=1572509331254:221:30)
    at TinyEmitter.emit (https://bayes.colorado.edu/continuous-testing/snapshot-1572495798551/axon/js/TinyEmitter.js?bust=1572509331254:68:55)
    at NumberProperty._notifyListeners (https://bayes.colorado.edu/continuous-testing/snapshot-1572495798551/axon/js/Property.js?bust=1572509331254:265:27)
    at NumberProperty.set (https://bayes.colorado.edu/continuous-testing/snapshot-1572495798551/axon/js/Property.js?bust=1572509331254:176:16)
id: Bayes Chrome
Approximately 10/30/2019, 10:23:18 PM
@chrisklus chrisklus mentioned this issue Jun 1, 2023
Open
chrisklus added a commit that referenced this issue Nov 11, 2019
@chrisklus
Convert playAreas to not change currentNumber with internal Propertie…
92062cd 
…s, make recreated paper ones not pickable, add guards when setting/reading currentNumberProperty, see #2, #4, #5, #6
@chrisklus
Copy link
Contributor

Thanks @KatieWoe. This is not showing up on CT anymore per my temporary fix in phetsims/number-suite-common#72, closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@chrisklus chrisklus

Labels
type:automated-testing
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@chrisklus @KatieWoe