-
-
Notifications
You must be signed in to change notification settings - Fork 35.3k
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
GLTFLoader: Vertex color alpha support #21533
Comments
That's exciting! :)
No, the logic for setting |
And we don't hardcode accessor type for vertex color attribute but we refer to accessor.type to determine the type (vec3, vec4, or so on) so vec4 vertex color attribute should already be supported. So, my understanding is vertex color alpha + glTF loader should be workable even without any change in the loader if Three.js core supports it, shouldn't it? |
Sounds good! |
I guess it'd be nice to have a new gltf model test. Right now they only have this one: |
/cc @emackey |
Is your feature request related to a problem? Please describe.
Now that we support vertex colors with alpha (#20975, #21530) how should
GLTFLoader
handle it?Describe the solution you'd like
I can see that it's supposed to be supported:
https://github.com/KhronosGroup/glTF/tree/master/specification/2.0
But should the loader automatically set
transparent: true
in the material when we find aVEC4
attribute?@donmccurdy @takahirox
The text was updated successfully, but these errors were encountered: