KhronosGroup / glTF

glTF – Runtime 3D Asset Delivery
Other
6.99k stars 1.13k forks source link

Fix reference BRDF implementation (#2386) #2392

Closed proog128 closed 1 week ago

proog128 commented 1 month ago

This PR fixes the bug with the squared metallic term in the reference BRDF implementation in Appendix B by rewriting the pseudocode according to the proposal of @lisyarus. In addition, all extensions referencing the code are updated.

Besides, I removed a few inconsistencies in the extensions: