KhronosGroup / glTF

glTF – Runtime 3D Asset Delivery
Other
7.19k stars 1.14k forks source link

KHR_draco_mesh_compression and triangle strips #2354

Open bghgary opened 11 months ago

bghgary commented 11 months ago

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

How should implementations handle this?

donmccurdy commented 8 months ago

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.