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

Mesh with tangents has validation errors and breaking compression #726

Closed
marwie opened this issue Apr 4, 2024 · 7 comments
Closed

Mesh with tangents has validation errors and breaking compression #726

marwie opened this issue Apr 4, 2024 · 7 comments
Assignees

Comments

@marwie
Copy link
Contributor

marwie commented Apr 4, 2024

Model exported with UnityGLTF has thousands of tangent validation error

draco compression fails with access out of memory exception for these models

model.zip

image

@hybridherbst
Copy link
Collaborator

@pfcDorn
Copy link
Contributor

pfcDorn commented Apr 5, 2024

Should not be related. The sign error was a Meta Avatar specific issue.
But i will take a look now.

@pfcDorn
Copy link
Contributor

pfcDorn commented Apr 5, 2024

Ok, after some investigation, i would say it's a Unity Draco bug. The Tangent support is new in draco, maybe there are still some issues. Currently i can't test it with gltFast, because it's not using the new com.unity.cloud.draco package (only this is supporting tangents)
I will contact Atteneder for this issue :)

@marwie
Copy link
Contributor Author

marwie commented Apr 5, 2024

I think got it for meshes with or without draco compression when exporting with UnityGLTF (validation errors for tangents in the resulting gltf)

@pfcDorn
Copy link
Contributor

pfcDorn commented Apr 5, 2024

Edit: no, Draco works fine. But he need to change the code documentation for proper use of the tangents. :D I found a hint in the draco changelog how to handle it correctly, now it works fine!

@marwie
Copy link
Contributor Author

marwie commented Apr 5, 2024

Sounds good - so export for draco compressed meshes is fixed?
I can't reproduce anymore with non-compressed meshes too so that would be the only issue

@pfcDorn
Copy link
Contributor

pfcDorn commented Apr 5, 2024

yeah, is fixed :)

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

No branches or pull requests

3 participants