Skip to content

Conversation

@pcwalton
Copy link
Contributor

@pcwalton pcwalton commented Dec 4, 2025

The current clustered decal code predates
PARTIALLY_BOUND_BINDING_ARRAY support in wgpu. As a result, only 8 clustered decals are allowed in the scene in order to avoid paying a high cost even if no decals of that type are present. This limitation makes Bevy's current clustered decal support insufficient for many use cases (e.g. bullet holes), so this patch increases the limit from 8 to 1024 if the wgpu features include PARTIALLY_BOUND_BINDING_ARRAY.

`PARTIALLY_BOUND_BINDING_ARRAY` `wgpu` feature is present.

The current clustered decal code predates
`PARTIALLY_BOUND_BINDING_ARRAY` support in `wgpu`. As a result, only 8
clustered decals are allowed in the scene in order to avoid paying a
high cost even if no decals of that type are present. This limitation
makes Bevy's current clustered decal support insufficient for many use
cases (e.g. bullet holes), so this patch increases the limit from 8 to
1024 if the `wgpu` features include `PARTIALLY_BOUND_BINDING_ARRAY`.
@pcwalton pcwalton added the A-Rendering Drawing game state to the screen label Dec 4, 2025
@pcwalton pcwalton requested a review from IceSentry December 4, 2025 16:51
Copy link
Contributor

@IceSentry IceSentry left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd like a comment explaining the 1024 limit but other than that LGTM

@pcwalton pcwalton added the S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it label Dec 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-Rendering Drawing game state to the screen S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

6 participants