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.

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

CT substantial total energy change after corrections #323

Open
KatieWoe opened this issue May 14, 2021 · 3 comments
Open

CT substantial total energy change after corrections #323

KatieWoe opened this issue May 14, 2021 · 3 comments

Comments

@KatieWoe
Copy link
Contributor

energy-skate-park : multitouch-fuzz : unbuilt
https://bayes.colorado.edu/continuous-testing/ct-snapshots/1620957325909/energy-skate-park/energy-skate-park_en.html?continuousTest=%7B%22test%22%3A%5B%22energy-skate-park%22%2C%22multitouch-fuzz%22%2C%22unbuilt%22%5D%2C%22snapshotName%22%3A%22snapshot-1620957325909%22%2C%22timestamp%22%3A1621005383599%7D&brand=phet&ea&fuzz&fuzzPointers=2&memoryLimit=1000&supportsPanAndZoom=false
Query: brand=phet&ea&fuzz&fuzzPointers=2&memoryLimit=1000&supportsPanAndZoom=false
Uncaught Error: Assertion failed: substantial total energy change after corrections
Error: Assertion failed: substantial total energy change after corrections
at window.assertions.assertFunction (https://bayes.colorado.edu/continuous-testing/ct-snapshots/1620957325909/assert/js/assert.js:25:13)
at MeasureModel.correctThermalEnergy (https://bayes.colorado.edu/continuous-testing/ct-snapshots/1620957325909/energy-skate-park/js/common/model/EnergySkateParkModel.js:584:15)
at MeasureModel.switchToGround (https://bayes.colorado.edu/continuous-testing/ct-snapshots/1620957325909/energy-skate-park/js/common/model/EnergySkateParkModel.js:525:35)
at MeasureModel.stepTrack (https://bayes.colorado.edu/continuous-testing/ct-snapshots/1620957325909/energy-skate-park/js/common/model/EnergySkateParkModel.js:1100:23)
at MeasureModel.stepModel (https://bayes.colorado.edu/continuous-testing/ct-snapshots/1620957325909/energy-skate-park/js/common/model/EnergySkateParkModel.js:1460:19)
at MeasureModel.stepModel (https://bayes.colorado.edu/continuous-testing/ct-snapshots/1620957325909/energy-skate-park/js/common/model/EnergySkateParkSaveSampleModel.js:161:32)
at MeasureModel.constantStep (https://bayes.colorado.edu/continuous-testing/ct-snapshots/1620957325909/energy-skate-park/js/common/model/EnergySkateParkModel.js:388:29)
at EventTimer.step (https://bayes.colorado.edu/continuous-testing/ct-snapshots/1620957325909/phet-core/js/EventTimer.js:120:12)
at MeasureModel.step (https://bayes.colorado.edu/continuous-testing/ct-snapshots/1620957325909/energy-skate-park/js/common/model/EnergySkateParkModel.js:437:21)
at https://bayes.colorado.edu/continuous-testing/ct-snapshots/1620957325909/joist/js/Sim.js:287:22
id: Bayes Chrome
Snapshot from 5/13/2021, 7:55:25 PM
@marlitas
Copy link
Contributor

We added more detail to the related assertion error, and will keep an eye out on CT since this is a rare hit.

@marlitas
Copy link
Contributor

The new logging looks like this:

Uncaught Error: Assertion failed: substantial total energy change after corrections. skaterTotalEnergy: -6.4853432765756605, correctedTotalEnergy: 0, delta: 6.4853432765756605

That seems buggy to me and really strange... I think more investigation is needed.

@marlitas
Copy link
Contributor

This is not within the scope of the upcoming character set publication. Marking as deferred.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants