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

add feature RG11B10UFLOAT_RENDERABLE on webgpu #3689

Merged
merged 2 commits into from
Apr 14, 2023

Conversation

mockersf
Copy link
Contributor

@mockersf mockersf commented Apr 14, 2023

Checklist

  • Run cargo clippy.
  • Run RUSTFLAGS=--cfg=web_sys_unstable_apis cargo clippy --target wasm32-unknown-unknown if applicable.
  • Add change to CHANGELOG.md. See simple instructions inside file.

Connections

Description

Add feature RG11B10UFLOAT_RENDERABLE on webgpu. I don't think this needs anything else added as, at least for Bevy, it was already working for metal/dx/vulkan

Testing

Rendered bloom examples from Bevy on webgpu and metal that use this texture format
webgpu:
Screenshot 2023-04-14 at 12 13 34

metal:
Screenshot 2023-04-14 at 12 31 11

@mockersf mockersf force-pushed the enable-feature-RG11B10UFLOAT_RENDERABLE branch from 6d34f07 to ad5e825 Compare April 14, 2023 10:42
@nical nical merged commit bf8e6fe into gfx-rs:trunk Apr 14, 2023
/// - Metal
///
/// This is a web and native feature.
const RG11B10UFLOAT_RENDERABLE = 1 << 23;
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 already working because bevy uses adapter specific texture format features. If a user isn't, it needs to both advertise this feature, and change how we check the flags for texture features

Copy link
Member

Choose a reason for hiding this comment

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

@mockersf you can use #3634 as an example for this

Copy link
Contributor

Choose a reason for hiding this comment

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

As wgpu 0.16 is going to be released in the next few days, if there is a need, I am happy to complete this part of the code.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I started looking into this, I will open the PR now but I'm trying to find if there are conditions to check before adding it to DX12/Vulkan/Metal

Copy link
Contributor Author

Choose a reason for hiding this comment

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

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.

5 participants