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 Interpolation attribute for the VertexOutput and add Error trait implementation for RenderError #49

Merged
merged 2 commits into from
Aug 6, 2023

Conversation

c-u
Copy link
Contributor

@c-u c-u commented Aug 6, 2023

Hi,
just two simple changes.

Without the interpolation attribute I got errors when compiling the shaders when running in a browser.

(User-defined vertex outputs and fragment inputs of scalar or vector integer type must always be specified as @interpolate(flat).)

from https://www.w3.org/TR/WGSL/#interpolation

I also noticed the missing impl Error with RenderError. If that was intentional feel free to revert.

Copy link
Owner

@grovesNL grovesNL left a comment

Choose a reason for hiding this comment

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

Oops, looks like wgpu is missing this validation. Thank you!

@grovesNL grovesNL merged commit 3c9f7c4 into grovesNL:main Aug 6, 2023
1 check passed
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