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

KHR_draco_mesh_compression and triangle strips #2354

Open
bghgary opened this issue Dec 12, 2023 · 1 comment
Open

KHR_draco_mesh_compression and triangle strips #2354

bghgary opened this issue Dec 12, 2023 · 1 comment

Comments

@bghgary
Copy link
Contributor

bghgary commented Dec 12, 2023

It seems like Draco does not support triangles strips: google/draco#423, but yet, the KHR_draco_mesh_compression spec explicitly states that it is supported.

How should implementations handle this?

@donmccurdy
Copy link
Contributor

donmccurdy commented Mar 6, 2024

It's not obvious to me what (if anything?) is missing for the sequential encoder to support triangle strips, but I gave it a quick test and hit generic errors. three.js does not do any special workaround here, and basically trusts that the vertex and index data returned by the decoder are what the glTF header says that they are. glTF Transform will skip Draco compression for any mode but plain triangles.

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

2 participants