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

many_cameras_lights should use same window settings as other stress tests #17183

Open
rparrett opened this issue Jan 6, 2025 · 0 comments
Open
Labels
A-Windowing Platform-agnostic interface layer to run your app in C-Bug An unexpected or incorrect behavior C-Examples An addition or correction to our examples D-Trivial Nice and easy! A great choice to get started with Bevy S-Ready-For-Implementation This issue is ready for an implementation PR. Go for it!

Comments

@rparrett
Copy link
Contributor

rparrett commented Jan 6, 2025

Bevy version

main

What you did

cargo run --example many_cameras_lights

What went wrong

Other stress tests

  • override the scale factor to 1.0 so we don't render at double resolution on some machines
  • set the resolution to a more common desktop resolution than the default
  • disable vsync so frame rate isn't capped

Additional info

This should be trivial, but making this change currently crashes the example due to #17188

@rparrett rparrett added C-Bug An unexpected or incorrect behavior S-Needs-Triage This issue needs to be labelled labels Jan 6, 2025
@BenjaminBrienen BenjaminBrienen added D-Trivial Nice and easy! A great choice to get started with Bevy C-Examples An addition or correction to our examples A-Windowing Platform-agnostic interface layer to run your app in S-Ready-For-Implementation This issue is ready for an implementation PR. Go for it! and removed S-Needs-Triage This issue needs to be labelled labels Jan 6, 2025
@rparrett rparrett changed the title many_cameras_lights stress test should use same resolution and scale factor as other stress tests many_cameras_lights should use same window settings as other stress tests Jan 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Windowing Platform-agnostic interface layer to run your app in C-Bug An unexpected or incorrect behavior C-Examples An addition or correction to our examples D-Trivial Nice and easy! A great choice to get started with Bevy S-Ready-For-Implementation This issue is ready for an implementation PR. Go for it!
Projects
None yet
Development

No branches or pull requests

2 participants