phetsims / circuit-construction-kit-common

"Circuit Construction Kit: Basics" is an educational simulation in HTML5, by PhET Interactive Simulations.
GNU General Public License v3.0
10 stars 10 forks source link

Migration: `Assertion failed: vertexNodeGroup.countProperty listener fired and array length differs, count=17, arrayLength=0` #997

Closed zepumph closed 9 months ago

zepumph commented 9 months ago

This has been on CT since at least late last week. I'm unsure if it is related to our solution over in https://github.com/phetsims/circuit-construction-kit-common/issues/996, but it may be. It fails in a case a simple as dragging out one circuit element and migrating. @samreid, let me know if you'd like to investigate together.

Full error:

``` circuit-construction-kit-dc-virtual-lab : migration : 1.3->main : assert http://128.138.93.172/continuous-testing/ct-snapshots/1702336416531/phet-io-wrappers/migration/?sim=circuit-construction-kit-dc-virtual-lab&oldVersion=1.3&phetioMigrationReport=assert&locales=*&phetioDebug=true&phetioWrapperDebug=true&fuzz&migrationRate=5000&&wrapperContinuousTest=%7B%22test%22%3A%5B%22circuit-construction-kit-dc-virtual-lab%22%2C%22migration%22%2C%221.3-%3Emain%22%2C%22assert%22%5D%2C%22snapshotName%22%3A%22snapshot-1702336416531%22%2C%22timestamp%22%3A1702337046715%7D Uncaught Error: Assertion failed: circuitConstructionKitDcVirtualLab.labScreen.view.circuitNode.vertexNodeGroup.countProperty listener fired and array length differs, count=17, arrayLength=0 Error: Assertion failed: circuitConstructionKitDcVirtualLab.labScreen.view.circuitNode.vertexNodeGroup.countProperty listener fired and array length differs, count=17, arrayLength=0 at window.assertions.assertFunction (http://128.138.93.172/continuous-testing/ct-snapshots/1702336416531/assert/js/assert.js:28:13) at assert (PhetioGroup.ts:91:16) at listener (TinyEmitter.ts:123:8) at emit (ReadOnlyProperty.ts:329:22) at _notifyListeners (ReadOnlyProperty.ts:377:46) at listener (PropertyStateHandler.ts:273:40) at attemptToApplyPhases (PropertyStateHandler.ts:193:11) at undeferAndNotifyProperties (PropertyStateHandler.ts:88:11) at listener (TinyEmitter.ts:123:8) at emit (Emitter.ts:62:21) [URL] http://128.138.93.172/continuous-testing/aqua/html/wrapper-test.html?url=..%2F..%2Fct-snapshots%2F1702336416531%2Fphet-io-wrappers%2Fmigration%2F%3Fsim%3Dcircuit-construction-kit-dc-virtual-lab%26oldVersion%3D1.3%26phetioMigrationReport%3Dassert%26locales%3D*%26phetioDebug%3Dtrue%26phetioWrapperDebug%3Dtrue%26fuzz%26migrationRate%3D5000%26&duration=80000&testInfo=%7B%22test%22%3A%5B%22circuit-construction-kit-dc-virtual-lab%22%2C%22migration%22%2C%221.3-%3Emain%22%2C%22assert%22%5D%2C%22snapshotName%22%3A%22snapshot-1702336416531%22%2C%22timestamp%22%3A1702337046715%7D [NAVIGATED] http://128.138.93.172/continuous-testing/aqua/html/wrapper-test.html?url=..%2F..%2Fct-snapshots%2F1702336416531%2Fphet-io-wrappers%2Fmigration%2F%3Fsim%3Dcircuit-construction-kit-dc-virtual-lab%26oldVersion%3D1.3%26phetioMigrationReport%3Dassert%26locales%3D*%26phetioDebug%3Dtrue%26phetioWrapperDebug%3Dtrue%26fuzz%26migrationRate%3D5000%26&duration=80000&testInfo=%7B%22test%22%3A%5B%22circuit-construction-kit-dc-virtual-lab%22%2C%22migration%22%2C%221.3-%3Emain%22%2C%22assert%22%5D%2C%22snapshotName%22%3A%22snapshot-1702336416531%22%2C%22timestamp%22%3A1702337046715%7D [NAVIGATED] about:blank [NAVIGATED] http://128.138.93.172/continuous-testing/ct-snapshots/1702336416531/phet-io-wrappers/migration/?sim=circuit-construction-kit-dc-virtual-lab&oldVersion=1.3&phetioMigrationReport=assert&locales=*&phetioDebug=true&phetioWrapperDebug=true&fuzz&migrationRate=5000&&wrapperContinuousTest=%7B%22test%22%3A%5B%22circuit-construction-kit-dc-virtual-lab%22%2C%22migration%22%2C%221.3-%3Emain%22%2C%22assert%22%5D%2C%22snapshotName%22%3A%22snapshot-1702336416531%22%2C%22timestamp%22%3A1702337046715%7D [NAVIGATED] about:blank [NAVIGATED] about:blank [CONSOLE] enabling assert [NAVIGATED] http://128.138.93.172/continuous-testing/ct-snapshots/1702336416531/studio/?sim=circuit-construction-kit-dc-virtual-lab&phetioWrapperDebug=true&wrapperContinuousTest=%7B%22test%22%3A%5B%22circuit-construction-kit-dc-virtual-lab%22%2C%22migration%22%2C%221.3-%3Emain%22%2C%22assert%22%5D%2C%22snapshotName%22%3A%22snapshot-1702336416531%22%2C%22timestamp%22%3A1702337046715%7D&phetioMigrationReport=assert&fuzz&locales=* [NAVIGATED] about:blank [CONSOLE] enabling assert [NAVIGATED] about:blank [NAVIGATED] http://128.138.93.172/continuous-testing/ct-snapshots/1702336416531/circuit-construction-kit-dc-virtual-lab/circuit-construction-kit-dc-virtual-lab_en.html?brand=phet-io&ea&debugger&postMessageOnError&sim=circuit-construction-kit-dc-virtual-lab&phetioWrapperDebug=true&wrapperContinuousTest=%7B%22test%22%3A%5B%22circuit-construction-kit-dc-virtual-lab%22%2C%22migration%22%2C%221.3-%3Emain%22%2C%22assert%22%5D%2C%22snapshotName%22%3A%22snapshot-1702336416531%22%2C%22timestamp%22%3A1702337046715%7D&phetioMigrationReport=assert&fuzz&locales=*&phetioEmitAPIBaseline&phetioCreateArchetypes&phetioEmitHighFrequencyEvents=false [CONSOLE] enabling assert [NAVIGATED] https://phet-io.colorado.edu/sims/circuit-construction-kit-dc-virtual-lab/1.3/wrappers/studio/?phetioWrapperDebug=true&wrapperContinuousTest=%7B%22test%22%3A%5B%22circuit-construction-kit-dc-virtual-lab%22%2C%22migration%22%2C%221.3-%3Emain%22%2C%22assert%22%5D%2C%22snapshotName%22%3A%22snapshot-1702336416531%22%2C%22timestamp%22%3A1702337046715%7D&phetioMigrationReport=assert&fuzz&exposeStandardPhetioWrapper [NAVIGATED] about:blank [CONSOLE] enabling assert [NAVIGATED] https://phet-io.colorado.edu/sims/circuit-construction-kit-dc-virtual-lab/1.3/circuit-construction-kit-dc-virtual-lab_all_phet-io_debug.html?ea&postMessageOnError&phetioWrapperDebug=true&wrapperContinuousTest=%7B%22test%22%3A%5B%22circuit-construction-kit-dc-virtual-lab%22%2C%22migration%22%2C%221.3-%3Emain%22%2C%22assert%22%5D%2C%22snapshotName%22%3A%22snapshot-1702336416531%22%2C%22timestamp%22%3A1702337046715%7D&phetioMigrationReport=assert&fuzz&exposeStandardPhetioWrapper&phetioEmitAPIBaseline&phetioCreateArchetypes&phetioEmitHighFrequencyEvents=false [CONSOLE] enabling assert [CONSOLE] an attempt was made to set the output level for a sound category on an uninitialized sound manager, ignoring [CONSOLE] an attempt was made to set the output level for a sound category on an uninitialized sound manager, ignoring [CONSOLE] continuous-test-wrapper-load [CONSOLE] Assertion failed: circuitConstructionKitDcVirtualLab.labScreen.view.circuitNode.vertexNodeGroup.countProperty listener fired and array length differs, count=17, arrayLength=0 [PAGE ERROR] Error: Error: Assertion failed: circuitConstructionKitDcVirtualLab.labScreen.view.circuitNode.vertexNodeGroup.countProperty listener fired and array length differs, count=17, arrayLength=0 at window.assertions.assertFunction (http://128.138.93.172/continuous-testing/ct-snapshots/1702336416531/assert/js/assert.js:28:13) at http://128.138.93.172/continuous-testing/ct-snapshots/1702336416531/chipper/dist/js/tandem/js/PhetioGroup.js:65:17 at TinyProperty.emit (http://128.138.93.172/continuous-testing/ct-snapshots/1702336416531/chipper/dist/js/axon/js/TinyEmitter.js:96:9) at NumberProperty._notifyListeners (http://128.138.93.172/continuous-testing/ct-snapshots/1702336416531/chipper/dist/js/axon/js/ReadOnlyProperty.js:250:23) at PhaseCallback.listener (http://128.138.93.172/continuous-testing/ct-snapshots/1702336416531/chipper/dist/js/axon/js/ReadOnlyProperty.js:295:47) at PropertyStateHandler.attemptToApplyPhases (http://128.138.93.172/continuous-testing/ct-snapshots/1702336416531/chipper/dist/js/axon/js/PropertyStateHandler.js:220:41) at PropertyStateHandler.undeferAndNotifyProperties (http://128.138.93.172/continuous-testing/ct-snapshots/1702336416531/chipper/dist/js/axon/js/PropertyStateHandler.js:149:12) at http://128.138.93.172/continuous-testing/ct-snapshots/1702336416531/chipper/dist/js/axon/js/PropertyStateHandler.js:54:12 at TinyEmitter.emit (http://128.138.93.172/continuous-testing/ct-snapshots/1702336416531/chipper/dist/js/axon/js/TinyEmitter.js:96:9) at Emitter.emit (http://128.138.93.172/continuous-testing/ct-snapshots/1702336416531/chipper/dist/js/axon/js/Emitter.js:51:22) [PAGE ERROR] Error: Error: Assertion failed: circuitConstructionKitDcVirtualLab.labScreen.view.circuitNode.vertexNodeGroup.countProperty listener fired and array length differs, count=17, arrayLength=0 at window.assertions.assertFunction (http://128.138.93.172/continuous-testing/ct-snapshots/1702336416531/assert/js/assert.js:28:13) at http://128.138.93.172/continuous-testing/ct-snapshots/1702336416531/chipper/dist/js/tandem/js/PhetioGroup.js:65:17 at TinyProperty.emit (http://128.138.93.172/continuous-testing/ct-snapshots/1702336416531/chipper/dist/js/axon/js/TinyEmitter.js:96:9) at NumberProperty._notifyListeners (http://128.138.93.172/continuous-testing/ct-snapshots/1702336416531/chipper/dist/js/axon/js/ReadOnlyProperty.js:250:23) at PhaseCallback.listener (http://128.138.93.172/continuous-testing/ct-snapshots/1702336416531/chipper/dist/js/axon/js/ReadOnlyProperty.js:295:47) at PropertyStateHandler.attemptToApplyPhases (http://128.138.93.172/continuous-testing/ct-snapshots/1702336416531/chipper/dist/js/axon/js/PropertyStateHandler.js:220:41) at PropertyStateHandler.undeferAndNotifyProperties (http://128.138.93.172/continuous-testing/ct-snapshots/1702336416531/chipper/dist/js/axon/js/PropertyStateHandler.js:149:12) at http://128.138.93.172/continuous-testing/ct-snapshots/1702336416531/chipper/dist/js/axon/js/PropertyStateHandler.js:54:12 at TinyEmitter.emit (http://128.138.93.172/continuous-testing/ct-snapshots/1702336416531/chipper/dist/js/axon/js/TinyEmitter.js:96:9) at Emitter.emit (http://128.138.93.172/continuous-testing/ct-snapshots/1702336416531/chipper/dist/js/axon/js/Emitter.js:51:22) [CONSOLE] continuous-test-wrapper-error id: "Sparky Node Puppeteer" Snapshot from 12/11/2023, 4:13:36 PM ```
zepumph commented 9 months ago

Duplicate of https://github.com/phetsims/circuit-construction-kit-dc/issues/203