Simplify Draco and Meshopt compression implementations
Opened this issue · 0 comments
donmccurdy commented
Meshopt
Currently glTF Transform's Meshopt implementation has its own accessor writing method, and doesn't support VertexLayout options. I think possibly glTF Transform could just write Meshopt-compatible buffer view layouts by default, and simplify the Meshopt implementation accordingly.
Draco
The Draco implementation currently has to duplicate accessors in rare situations, leading to complex bugs. It may be worth a change to require pre-processing with draco(), and failing if Draco compression is attempted when that pre-processing has not been applied. Or other simplifications may be possible.