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

[dx12] Use typeless formats for textures that might be viewed as srgb or non-srgb #3555

Merged
merged 1 commit into from
Mar 3, 2023

Conversation

teoxoy
Copy link
Member

@teoxoy teoxoy commented Mar 3, 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
View formats initially implemented in #3237

Description
Use typeless formats for textures that might be viewed as srgb or non-srgb.
I also tried to query CastingFullyTypedFormatSupported and skip the typeless formats but later realized that winapi doesn't support options3.

Testing
I don't have the hardware to properly test this.

@teoxoy teoxoy added this to the WebGPU Specification V1 milestone Mar 3, 2023
@teoxoy teoxoy force-pushed the dx12-typeless-view-formats branch 2 times, most recently from 8a6479f to 025af7d Compare March 3, 2023 15:58
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 minus comment

wgpu-hal/src/dx12/adapter.rs Outdated Show resolved Hide resolved
… or non-srgb

query `CastingFullyTypedFormatSupported` and skip typeless formats if available
@teoxoy teoxoy force-pushed the dx12-typeless-view-formats branch from 025af7d to 4507a2b Compare March 3, 2023 19:32
@teoxoy teoxoy requested a review from cwfitzgerald March 3, 2023 19:32
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.

Nice

@cwfitzgerald cwfitzgerald enabled auto-merge (squash) March 3, 2023 19:33
@cwfitzgerald cwfitzgerald merged commit 54e4c86 into gfx-rs:master Mar 3, 2023
@teoxoy teoxoy deleted the dx12-typeless-view-formats branch March 3, 2023 19:46
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.

2 participants