Skip to content

Conversation

@dydavidkim
Copy link

To see the logs from the Cloud Build CI, please join either our discussion or announcement mailing list.

Linchenn and others added 4 commits September 28, 2022 17:26
Fix #6822

Problem
1: On some GPUs, even if a and b are both non-NaN, the value of isNaN in vec4 isNaN = min(vec4(isnan(a)) + vec4(isnan(b)), vec4(1.0)); are still larger than 0., which misleads all values become NAN.
2: After resolving NAN issue, the result is still incorrect. It seems that the isnan_custom is not well supported on the problem GPU. After switching back to builtin isnan, everything works well.

Solution:
Use the bool type bvec4 instead of float type vec4 to calculate isNaN to avoid the the float precision issue when comparing with zero.
Meanwhile, add an env flag WEBGL2_ISNAN_CUSTOM to allow user to specify which isnan to use.
* Upgrade nodejs to 18.7.0

* Fix hash table test string not passed as base64
@dydavidkim dydavidkim merged commit 9d09b67 into CodeSmithDSMLProjects:resizing Sep 29, 2022
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.

5 participants