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 cannot be decoded after sequential encoding #1013

Closed
donmccurdy opened this issue Jul 9, 2023 · 3 comments
Closed

Mesh cannot be decoded after sequential encoding #1013

donmccurdy opened this issue Jul 9, 2023 · 3 comments

Comments

@donmccurdy
Copy link
Contributor

I have a user-reported error related to a file that becomes invalid (the decoder always fails) after Draco encoding with the sequential method. I've stripped everything unrelated from the file, so it contains a only single mesh primitive in a basic scene:

File: PrimSimple.zip

Draco compression in glTF Transform or glTF Pipeline gives the same result – edgebreaker compression yields a valid file, and sequential compression yields a broken one. Files were tested in multiple engines, and none could decode the broken version.

The original file includes morph targets, so while edgebreaker is a workaround in this simplified case, it's not an option for the source asset. Related: #929 (comment).

Example with glTF Transform CLI (https://gltf-transform.dev/cli) –

gltf-transform draco PrimSimple.gltf PrimSimple_draco_seq.glb --method sequential
gltf-transform inspect PrimSimple_draco_seq.glb 
# error!
@ondys
Copy link
Collaborator

ondys commented Oct 27, 2023

this will be fixed in the next release.

@hybridherbst
Copy link

@ondys is this fixed in a current release already? If so, which?

@ondys
Copy link
Collaborator

ondys commented Feb 2, 2024

Yes this should be fixed in the 1.5.7 release

@ondys ondys closed this as completed Feb 2, 2024
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