-
Notifications
You must be signed in to change notification settings - Fork 143
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
Sampler filter #7
Conversation
…d texture. Generated mipmap for texture. Added cleanup code. Added animationFrame
… only instead of each new unit
6548d04
to
801fe2c
Compare
801fe2c
to
9ddb18e
Compare
@pjcozzi Ready to merge |
@shrekshao could you check this one actually |
b5315f9
to
63a9456
Compare
@pjcozzi Should be ready now |
Made some style tweaks in f986ade. I'll comment on a few lines of interest. |
<html lang="en"> | ||
|
||
<head> | ||
<title>WebGL 2 Samples - sampler_filter</title> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Always use "WebGL", not "webgl."
<div id="info">WebGL 2 Samples - sampler_filter</div> | ||
|
||
<script id="vs" type="x-shader/x-vertex"> | ||
#version 300 es |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A while back I changed the helper function so #version
can be on a new line like this instead of on the same line as the script
tag.
Just those comments. This looks really good, and is close to being ready. |
Thanks @pjcozzi ! Going to fix these now |
@pjcozzi I went through the comments and updated with very small tweaks. |
This is a masterpiece. |
This sample showcase different filters for samplers