Add many_materials stress test #17346
Conversation
|
The generated |
IQuick143
added
A-Rendering
- Removed ground plane as it was unnecessary. - Adjusted camera position for a better view. - Reused mesh handle to improve performance. - Regenerated examples README using `cargo run -p build-templated-pages -- update examples`.
|
I've addressed the feedback in a new commit. Please take another look when you have time. Thanks! |
|
Re-tested with the changes. Looks good! |
BenjaminBrienen
added
S-Needs-SME
|
|
||
| // Cubes | ||
| let mesh_handle = mesh_assets.add(Cuboid::from_size(Vec3::ONE)); | ||
| for x in 0..n { |
There was a problem hiding this comment.
We might need to make this 3 dimensional if frustum culling interacts with this.
There was a problem hiding this comment.
actually, nevermind, I didn't realize the camera was adjusted to fit everything on screen when I made that comment.
| } | ||
| } | ||
|
|
||
| fn animate_materials( |
There was a problem hiding this comment.
It might be good to have a runtime toggle for this. Like pressing spacebar.
This way we can more easily confirm that it's the animation that is the issue and not rendering.
IceSentry
added
S-Ready-For-Final-Review
Objective
many_materialsto benchmark the rendering performance of many animated materials.Solution
examples/stress_tests/many_materials.rs) that renders a grid of cubes with animated materials.-ncommand-line argument (or--grid-size). The default grid size is 10x10.Testing
Showcase
Click to view showcase