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

Assertions failing in debug sim #20

Closed
KatieWoe opened this issue Mar 10, 2023 · 3 comments
Closed

Assertions failing in debug sim #20

KatieWoe opened this issue Mar 10, 2023 · 3 comments
Assignees
Labels
type:bug Something isn't working

Comments

@KatieWoe
Copy link

Test device
Samsung
Operating System
Win 11
Browser
Chrome
Problem description
For phetsims/qa#917
When running a fuzz test on the debug version the sim this assertion appeared several times.

Visuals
keepsshowingup

Troubleshooting information:

!!!!! DO NOT EDIT !!!!!
Name: ‪Number Compare‬
URL: https://phet-dev.colorado.edu/html/number-compare/1.0.0-dev.4/phet/number-compare_all_phet.html
Version: 1.0.0-dev.4 2023-03-09 16:37:22 UTC
Features missing: applicationcache, applicationcache, touch
Flags: pixelRatioScaling
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/110.0.0.0 Safari/537.36
Language: en-US
Window: 1536x714
Pixel Ratio: 1.25/1
WebGL: WebGL 1.0 (OpenGL ES 2.0 Chromium)
GLSL: WebGL GLSL ES 1.0 (OpenGL ES GLSL ES 1.0 Chromium)
Vendor: WebKit (WebKit WebGL)
Vertex: attribs: 16 varying: 31 uniform: 4096
Texture: size: 8192 imageUnits: 32 (vertex: 32, combined: 64)
Max viewport: 8192x8192
OES_texture_float: true
Dependencies JSON: {}

@KatieWoe KatieWoe added the type:bug Something isn't working label Mar 10, 2023
chrisklus added a commit to phetsims/number-suite-common that referenced this issue Mar 15, 2023
chrisklus added a commit to phetsims/number-play that referenced this issue Mar 15, 2023
@chrisklus
Copy link
Contributor

Thanks @KatieWoe! I think this was actually the cause of phetsims/number-suite-common#55, should be fixed in the above commits. Leaving self assigned to verify in the new dev version.

@chrisklus
Copy link
Contributor

I am not seeing this in dev.6, closing.

@chrisklus
Copy link
Contributor

The problem was the string to be read out for testing a voice in preferences was being stripped out during a build. @samreid helped me added a reference to it so that would not happen.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants