-
Notifications
You must be signed in to change notification settings - Fork 168
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 EXT_texture_filter_anisotropic extension #203
Conversation
Thank you for contributing another extension. Overall this looks good and tests pass in CI on both Linux and macOS. Note: on Linux, the number of tests run hasn't increased, and remains at 16137. On macOS, the number of tests run has increased from 16177 to 16197 - about 20. This suggests that this extension isn't available on Linux in CI. I couldn't find "anisotropic" in the list of extensions from the CI Linux debug output: Linux GL debug output
That being said, while it passes CI, I'm having problems locally. On my iMac (macOS Catalina, AMD Radeon Pro 570 4 GB), at least one test fails:
This happens on this line of the tests: https://github.com/stackgl/gl-conformance/blob/cfb4649b21cd138c3a6870d4534422287e054d3f/conformance-suites/1.0.3/conformance/extensions/ext-texture-filter-anisotropic.html#L177 Or, on line 2633 in the gl-conformance package's extensions_ext-texture-filter-anisotropic.js file. Running this page on my machine makes it pass https://www.khronos.org/registry/webgl/conformance-suites/1.0.3/conformance/extensions/ext-texture-filter-anisotropic.html?webglVersion=1 Any idea why this would be happening? |
Did you rebuild C++ code? I had the same issue before I updated C++ code to use float overloads instead of default integer ones |
Oh! Good point. I forgot that you made changes to the native code. Let me try. |
All tests pass! Fantastic. I'll do a final code review later this evening, merge it in, and release to npm. |
Prefer if blocks to simply be handled outside of the switch case, if possible.
Released as 4.8.0 |
No description provided.