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

feat(wgsl): add first and either sampling for @interpolate(flat, …) #6181

Conversation

ErichDonGubler
Copy link
Member

@ErichDonGubler ErichDonGubler commented Aug 29, 2024

Connections

Resolves #6122.

Description

Context is in the linked issue.

Testing

Added some coverage for both (1) new valid parses as snapshot testing and (2) invalid parses as validation testing.

Checklist

  • Run cargo fmt.
  • Run cargo clippy. If applicable, add:
    • --target wasm32-unknown-unknown
    • --target wasm32-unknown-emscripten
  • Run cargo xtask test to run tests.
  • Add change to CHANGELOG.md. See simple instructions inside file.

@ErichDonGubler ErichDonGubler self-assigned this Aug 29, 2024
@ErichDonGubler ErichDonGubler added naga Shader Translator lang: WGSL WebGPU Shading Language labels Aug 29, 2024
@ErichDonGubler ErichDonGubler force-pushed the first-or-either-sampling-for-flat-interp branch from c8831c5 to 6c1ef9f Compare August 29, 2024 03:02
@ErichDonGubler ErichDonGubler changed the title feat(wgsl): add first and either sampling types [wgsl-in] Add first and either sampling types for flat interpolation Aug 29, 2024
@ErichDonGubler ErichDonGubler changed the title [wgsl-in] Add first and either sampling types for flat interpolation feat(wgsl): add first and either sampling types for flat interpolation Aug 29, 2024
@ErichDonGubler ErichDonGubler marked this pull request as ready for review August 29, 2024 11:54
@ErichDonGubler ErichDonGubler requested review from a team as code owners August 29, 2024 11:54
@ErichDonGubler ErichDonGubler changed the title feat(wgsl): add first and either sampling types for flat interpolation feat(wgsl): add first and either sampling types for @interpolate(flat, …) Aug 29, 2024
@ErichDonGubler ErichDonGubler force-pushed the first-or-either-sampling-for-flat-interp branch 2 times, most recently from eb2521a to ec3f47c Compare August 29, 2024 12:24
naga/src/back/glsl/mod.rs Outdated Show resolved Hide resolved
@ErichDonGubler ErichDonGubler force-pushed the first-or-either-sampling-for-flat-interp branch from ec3f47c to 0591c9a Compare August 30, 2024 14:53
@ErichDonGubler ErichDonGubler changed the title feat(wgsl): add first and either sampling types for @interpolate(flat, …) feat(wgsl): add first and either sampling for @interpolate(flat, …) Aug 30, 2024
@ErichDonGubler ErichDonGubler force-pushed the first-or-either-sampling-for-flat-interp branch from 0591c9a to 5c69262 Compare August 30, 2024 14:55
@ErichDonGubler ErichDonGubler enabled auto-merge (squash) August 30, 2024 14:55
@ErichDonGubler ErichDonGubler merged commit 04618b3 into gfx-rs:trunk Aug 30, 2024
25 checks passed
@ErichDonGubler ErichDonGubler deleted the first-or-either-sampling-for-flat-interp branch August 30, 2024 15:40
@ErichDonGubler ErichDonGubler mentioned this pull request Sep 4, 2024
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lang: WGSL WebGPU Shading Language naga Shader Translator
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Support first and either sampling in @interpolate(flat, …)
2 participants