Skip to content
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

Shader compilation error on r132 #809

Closed
0b5vr opened this issue Sep 10, 2021 · 2 comments · Fixed by #811
Closed

Shader compilation error on r132 #809

0b5vr opened this issue Sep 10, 2021 · 2 comments · Fixed by #811
Assignees
Labels
bug Something isn't working
Milestone

Comments

@0b5vr
Copy link
Contributor

0b5vr commented Sep 10, 2021

https://glitch.com/edit/#!/noisy-big-value?path=index.html%3A11%3A44

THREE.WebGLProgram: Shader Error 1282 - VALIDATE_STATUS false

Program Info Log: Fragment shader is not compiled.


FRAGMENT

ERROR: 0:895: 'BRDF_Diffuse_Lambert' : no matching overloaded function found
ERROR: 0:1154: 'getDirectionalDirectLightIrradiance' : no matching overloaded function found
ERROR: 0:1190: 'BRDF_Diffuse_Lambert' : no matching overloaded function found
@0b5vr 0b5vr added the bug Something isn't working label Sep 10, 2021
@0b5vr 0b5vr added this to the next milestone Sep 10, 2021
@0b5vr 0b5vr self-assigned this Sep 10, 2021
@0b5vr
Copy link
Contributor Author

0b5vr commented Sep 10, 2021

Kinda unrelated but it seems .skinning .morphTargets .morphNormals are no longer required since it can be inferred from mesh, starting from r131:
mrdoob/three.js#22169
mrdoob/three.js#22166

@0b5vr
Copy link
Contributor Author

0b5vr commented Sep 10, 2021

Kinda unrelated but it seems .skinning .morphTargets .morphNormals are no longer required since it can be inferred from mesh, starting from r131:
mrdoob/three.js#22169
mrdoob/three.js#22166

.skinning was starting from r129 mrdoob/three.js#21788

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant