Skip to content

Update wgpu to 0.15#1789

Merged
hecrj merged 4 commits intoiced-rs:masterfrom
ElhamAryanpur:master
Apr 13, 2023
Merged

Update wgpu to 0.15#1789
hecrj merged 4 commits intoiced-rs:masterfrom
ElhamAryanpur:master

Conversation

@ElhamAryanpur
Copy link
Contributor

Hey there!

This is my first time contributing to this amazing project! I was actually trying to add iced as a plugin to my [engine][https://github.com/AryanpurTech/BlueEngine] and I noticed that the wgpu was a version behind, even though it's been a while since it was released, so thought to bump the versions and update the code related to them.

Please review and let me know if it's ok or if I need to update some parts,
thanks,

@hecrj hecrj added improvement An internal improvement rendering wgpu labels Apr 13, 2023
@hecrj hecrj added this to the 0.9.0 milestone Apr 13, 2023
@hecrj hecrj changed the title Updated wgpu to 0.15 in iced_wgpu and integration_wgpu Update wgpu to 0.15 Apr 13, 2023
Copy link
Member

@hecrj hecrj left a comment

Choose a reason for hiding this comment

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

Thank you! 🎉

I changed some things here and there, but everything looks good to go!

capabilities
.formats
.iter()
.filter(|format| format.describe().srgb)
Copy link
Member

Choose a reason for hiding this comment

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

This is cool! Hopefully it fixes inaccurate colors for some!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Haha yeah the API is improving a lot, I was surprised too! Thank you.

@hecrj hecrj enabled auto-merge April 13, 2023 04:06
@hecrj hecrj merged commit cf35c85 into iced-rs:master Apr 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement An internal improvement rendering wgpu

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments