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

Split Blendability and Filterability into Two Different TextureFormatFeatureFlags #3012

Merged

Conversation

stakka
Copy link
Contributor

@stakka stakka commented Sep 5, 2022

…FeatureFlags

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
Split Blendability and Filterability into Two Different TextureFormatFeatureFlags #2943

Description
By splitting up Blendability and Filterablity in the texture format feature flags, it is now possible to elude the limitation enforced by WebGPU to have a filterable texture format when blending is set on the color state of the render pipeline by using Features::TEXTURE_ADAPTER_SPECIFIC_FORMAT_FEATURES.

Testing
Tested on
backend = Metal
iOS: 15.6 (both real device and simulator)
device: iPad Pro 2021 12.9"
using the feature flag and the RG32Float format according to the issue described here.

Copy link
Member

@cwfitzgerald cwfitzgerald left a comment

Choose a reason for hiding this comment

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

Looks good outside of a comment

wgpu-core/src/device/mod.rs Outdated Show resolved Hide resolved
Copy link
Member

@cwfitzgerald cwfitzgerald left a comment

Choose a reason for hiding this comment

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

Dope!

@cwfitzgerald cwfitzgerald enabled auto-merge (squash) September 19, 2022 17:56
@cwfitzgerald
Copy link
Member

Just needs a formatting pass and we're g2g

auto-merge was automatically disabled September 20, 2022 08:24

Head branch was pushed to by a user without write access

@cwfitzgerald cwfitzgerald enabled auto-merge (squash) September 21, 2022 19:21
@cwfitzgerald cwfitzgerald merged commit 2c1d7a8 into gfx-rs:master Sep 21, 2022
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