donmccurdy / glTF-Transform

glTF 2.0 SDK for JavaScript and TypeScript, on Web and Node.js.
https://gltf-transform.dev
MIT License
1.3k stars 145 forks source link

fix(functions): Group meshopt-compressed accessors by parent #1384

Closed donmccurdy closed 2 months ago

donmccurdy commented 2 months ago

By default, glTF Transform groups accessors into buffer views by their parent mesh primitives. Currently the Meshopt encoding implementation does not do that grouping, and for consistency it should. If there's a compelling reason to allow buffer views to be unified rather than grouped, that can potentially be added as a new option on the I/O class that isn't specific to Meshopt.