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

switch bevy_image to use wgpu-types wherever possible instead of wgpu #16620

Merged
merged 2 commits into from
Dec 3, 2024

Conversation

atlv24
Copy link
Contributor

@atlv24 atlv24 commented Dec 3, 2024

Objective

  • dont depend on wgpu if we dont have to

Solution

  • works towards this, but doesnt fully accomplish it. the remaining types stopping us from doing this need to be moved upstream, i will PR this

Testing

  • 3d_scene runs

@atlv24
Copy link
Contributor Author

atlv24 commented Dec 3, 2024

Those 3 are the only types which need moving. Once these merge we will be able to only pull in wgpu-types for bevy_image and bevy_mesh (and soon bevy_camera)

@atlv24
Copy link
Contributor Author

atlv24 commented Dec 3, 2024

All PRs have been merged, this will be good to go next wgpu release.

@@ -49,6 +49,7 @@ image = { version = "0.25.2", default-features = false }
# misc
bitflags = { version = "2.3", features = ["serde"] }
bytemuck = { version = "1.5" }
wgpu-types = { version = "23", default-features = false }
wgpu = { version = "23.0.1", default-features = false }
Copy link
Contributor

Choose a reason for hiding this comment

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

Reminder to remember to remove this

Copy link
Contributor Author

Choose a reason for hiding this comment

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

can't remove it this PR, but once we upgrade wgpu we can

Co-authored-by: François Mockers <[email protected]>
@mockersf mockersf added this pull request to the merge queue Dec 3, 2024
Merged via the queue into bevyengine:main with commit d4883a9 Dec 3, 2024
31 of 32 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants