-
Notifications
You must be signed in to change notification settings - Fork 476
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
Update glTF specific shader #63
Comments
@sbtron I wonder what's the purpose of using a specific glTF shader instead of the Unity's Standard one ? Also, just to let you know, we are working on a "Design-time import of glTF 2.0 files" from this project and I guess it could be a good contribution, as we are facing and solving a few of the issues. I just don't know a lot yet about the rules and how our choices/solutions are ok with the rest of the plugin. |
It's more than that. Unity doesn't multiply the colors against the texture where as glTF does. Unity doesn't support texture coordinate sets where as glTF does. I think we need to think about supporting both. |
Ok thanks @bghgary . +1 for the texture coordinate sets (but colors are multiplied against texture if I'm right). |
Sorry, yeah, it is multiplied if there is a color in addition to texture, but some properties don't behave this way. For example, metallic factor in glTF is not representable in Unity's standard material if metallic texture is also specified. Another example is for the spec-gloss extension where Unity expects only one of specular color or texture but not both. |
This is done. |
Current one doesn't work on Android and was stripped out.
Need to figure out a better solution.
The text was updated successfully, but these errors were encountered: