-
Notifications
You must be signed in to change notification settings - Fork 305
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
wgsl: add refract as a built-in function #1901
Conversation
Add the refract into WGSL, which is already a built in function in GLSL, HLSL and MSL. Also correct a mistyping in the description of reflect.
A problem is that the Preview PR action fails saying that "HTTP Error: 429, Couldn't create channel on |
Please take a look, thanks! @dneto0 @kvark @kainino0x @Kangz |
I still get the |
I think this is non-controversial enough that we can merge now. |
See: gpuweb/gpuweb#1901 Bug: tint:950 Change-Id: I6f00ab753a2ddf2374352ddf636e1abfebe86ba7 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/56777 Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: Ben Clayton <bclayton@google.com> Reviewed-by: Antonio Maiorano <amaiorano@google.com>
Add the refract into WGSL, which is already a built in function in GLSL, HLSL and MSL. Please see #1087 (comment).
Also correct a mistyping in the description of reflect.
A problem is that while GLSL said "The type of I and N (the first two operands) must be a scalar or vector with a floating-point component type", HLSL said they must be "vector", rather than "scale or vector". The situation is the same for reflect.