-
-
Notifications
You must be signed in to change notification settings - Fork 155
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
Draco: memory access out of bounds #928
Comments
I'm AFK for a bit, but do you mind testing whether this works with glTF Pipeline? I've seen this issue in the past and think it might be a Draco library issue, if it fails in glTF Pipeline as well that's a good sign it's Draco. |
Hi @donmccurdy it runs fine with gltf-pipeline |
I also tried with gltf-transform 3.2.1 again and get:
|
On MacOS the error message doesn't mention memory, but reports a "bus error". 🤔 I'm also noticing validation errors that look important here. Is it possible this could be an error in the UnityGLTF exporter? I'd be curious what glTF Pipeline is doing to handle NaN, I'm not sure we'd want to scrub through vertex data looking for those.
|
Hello @donmccurdy I tried exporting the obj to glb using Blender and that works fine with draco then - so it's most likely a UnityGLTF bug as you said. I should have checked the validation errors error, sorry. |
Cool, no worries! Glad the validator catches this stuff, not sure how we'd have discovered a small handful of NaNs in there otherwise. 😅 |
Just to close the loop, it's actually a Unity OBJ Importer bug 🥹, the importer produces a bunch of NaN tangents for that model. I've followed up with them and added some sanitization for exporting with UnityGltf. Thanks! |
Describe the bug
compressing the attached glb results in a
RuntimeError: memory access out of bounds
errorTo Reproduce
Steps to reproduce the behavior:
Expected behavior
Mesh should be compressed, no errors
Versions:
Additional context
MyMesh.zip
The text was updated successfully, but these errors were encountered: