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

[Merged by Bors] - Use uniform buffer usage for SkinnedMeshUniform instead of all usages #4816

Closed
wants to merge 2 commits into from

Conversation

teoxoy
Copy link
Contributor

@teoxoy teoxoy commented May 20, 2022

Objective

fixes #4811 (caused by #4339 [exact change] - where the buffer type has been changed from UniformVec to BufferVec)

Solution

Use uniform buffer usage for SkinnedMeshUniform instead of all usages due to the Default derive.

@james7132 james7132 added C-Bug An unexpected or incorrect behavior A-Rendering Drawing game state to the screen labels May 20, 2022
@alice-i-cecile alice-i-cecile added the P-Critical This must be fixed immediately or contributors or users will be severely impacted label May 20, 2022
@alice-i-cecile alice-i-cecile added this to the Bevy 0.8 milestone May 20, 2022
@alice-i-cecile alice-i-cecile added P-High This is particularly urgent, and deserves immediate attention and removed P-Critical This must be fixed immediately or contributors or users will be severely impacted labels May 20, 2022
@mockersf mockersf 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 May 20, 2022
Copy link
Member

@cart cart left a comment

Choose a reason for hiding this comment

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

This fixes it for me. Nice work!

@cart
Copy link
Member

cart commented May 20, 2022

bors r+

bors bot pushed a commit that referenced this pull request May 20, 2022
…es (#4816)

# Objective

fixes #4811 (caused by #4339 [[exact change](https://github.com/bevyengine/bevy/pull/4339/files#diff-4bf3ed03d4129aad9f5678ba19f9b14ee8e3e61d6f6365e82197b01c74468b10R712-R721)] - where the buffer type has been changed from `UniformVec` to `BufferVec`)

## Solution

Use uniform buffer usage for `SkinnedMeshUniform` instead of all usages due to the `Default` derive.
@bors bors bot changed the title Use uniform buffer usage for SkinnedMeshUniform instead of all usages [Merged by Bors] - Use uniform buffer usage for SkinnedMeshUniform instead of all usages May 20, 2022
@bors bors bot closed this May 20, 2022
@teoxoy teoxoy deleted the fix-all-usages branch May 20, 2022 22:24
james7132 pushed a commit to james7132/bevy that referenced this pull request Jun 7, 2022
…es (bevyengine#4816)

# Objective

fixes bevyengine#4811 (caused by bevyengine#4339 [[exact change](https://github.com/bevyengine/bevy/pull/4339/files#diff-4bf3ed03d4129aad9f5678ba19f9b14ee8e3e61d6f6365e82197b01c74468b10R712-R721)] - where the buffer type has been changed from `UniformVec` to `BufferVec`)

## Solution

Use uniform buffer usage for `SkinnedMeshUniform` instead of all usages due to the `Default` derive.
ItsDoot pushed a commit to ItsDoot/bevy that referenced this pull request Feb 1, 2023
…es (bevyengine#4816)

# Objective

fixes bevyengine#4811 (caused by bevyengine#4339 [[exact change](https://github.com/bevyengine/bevy/pull/4339/files#diff-4bf3ed03d4129aad9f5678ba19f9b14ee8e3e61d6f6365e82197b01c74468b10R712-R721)] - where the buffer type has been changed from `UniformVec` to `BufferVec`)

## Solution

Use uniform buffer usage for `SkinnedMeshUniform` instead of all usages due to the `Default` derive.
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 C-Bug An unexpected or incorrect behavior P-High This is particularly urgent, and deserves immediate attention S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Panic on startup in web build
5 participants