-
Notifications
You must be signed in to change notification settings - Fork 4
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
Assertion failed: parallel system is too tall #47
Comments
There have been no changes to this sim for a long time, so I suspect that a common code change is the culprit. I first tested with my working copy, which was pulled yesterday - no problem. I then ran pull-all.sh changed these common-code repos in my working copy: One of them is the culprit. |
I asked about this on the Slack developer channel. @zepumph pointed me to a11y changes in |
The breaking change was to HSlider in phetsims/sun@f309a64. The nature of the change and reason for the break was discussed in Slack:
|
Breaking changes were also made to
|
Issue that breaking changes were made for: phetsims/scenery-phet#341 |
I’ve noted this issue in the commits that reverted HSlider and NumberControl. @phet-steele please verify after pulling sun and scenery-phet. |
Verified! |
Happens on load (master)
The text was updated successfully, but these errors were encountered: