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

UVs not Working for 2.0 branch #68

Closed
John-DeCorato-HZ opened this issue Aug 11, 2017 · 5 comments
Closed

UVs not Working for 2.0 branch #68

John-DeCorato-HZ opened this issue Aug 11, 2017 · 5 comments
Labels

Comments

@John-DeCorato-HZ
Copy link
Contributor

Title is self-explanatory. Here is a test file and the output.

test-cude.dae
test-cude.gltf

@pjcozzi pjcozzi added the bug label Aug 12, 2017
@lasalvavida
Copy link
Contributor

@John-DeCorato-HZ, the converter does optimize out unused texture coordinates to reduce the model size. Since your material contains no textures, they are removed.

@John-DeCorato-HZ
Copy link
Contributor Author

Hmm, we have a dynamic asset pipeline where we add textures separately, so we don't save them with our models. Is there a workaround besides adding dummy textures to everything?

@lasalvavida
Copy link
Contributor

So this is a pretty easy fix. I'll do it if I have time, but if someone else is willing to open a PR, this is a great beginner issue. The fix would entail adding something like a preserveUnusedSemantics boolean to COLLADA2GLTF::Options, defining the option in the argument parser and using the option to disable the call to GLTF::Asset::removeUnusedSemantics.

@John-DeCorato-HZ
Copy link
Contributor Author

Excellent. I compiled a version myself without removeUnusedSemantics already but that option would make future proofing easy.

@lasalvavida
Copy link
Contributor

Closed by #245

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants