-
-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
index out of bounds when running any Bevy example #6231
Comments
it seems to be running OK in CI so potentially a macOS bug |
Ping @VitalyAnkh for thoughts. |
Presumably this is only occurring for you on examples that invoke the renderer? Hello World for example shouldn't hit this path at all. |
I can't reproduce this on Window 10; I suspect this is Mac specific. |
reading the code, this will happen on iOS and android too. on those systems we don't create the window early, so the surface is not yet available when initialising the renderer |
The linked PR makes sense. Thanks! I wonder why the "run exampes" check didn't catch this problem. Could we make this check examples on more platforms/features? |
There is a balance with CI time and example coverage. At one point I wanted to try to mitigate this by running the examples occasionally in a separate repo with a dashboard: https://rparrett.github.io/prototype_bevy_example_runner/ This runs all examples, but only on Linux right now. |
Bevy CI is running a few examples in wasm on firefox and chrome, and in native on linux and windows. There may be a way to also run them on macOS, but I haven't had the time to set it up yet. Running on iOS and android would be great, but I'm not aware of it being possible in GitHub actions |
…evyengine#6233) # Objective - Fix bevyengine#6231 ## Solution - In case no supported format is found, try to use Bevy default instead of panicking
…evyengine#6233) # Objective - Fix bevyengine#6231 ## Solution - In case no supported format is found, try to use Bevy default instead of panicking
…evyengine#6233) # Objective - Fix bevyengine#6231 ## Solution - In case no supported format is found, try to use Bevy default instead of panicking
…evyengine#6233) # Objective - Fix bevyengine#6231 ## Solution - In case no supported format is found, try to use Bevy default instead of panicking
Bevy version
7673db7
Issue exists since #5413, and disappear if I revert that commit
[Optional] Relevant system information
What you did
The text was updated successfully, but these errors were encountered: