Arseny Kapoulkine

There’s an instancing example (SimpleInstancing) in glTF- Sample-Assets so I don’t know if we need a second one; not sure what the policy is wrt having examples in this repository....

Please let me know if any further changes are needed on this PR (or if there are no plans to merge it).

Yup, that's definitely the same bug.

One issue with using range_to_replace is that it results in odd behavior if the completion is a standalone action instead of a continuation of prefix input. E.g. if `he` is...

> I understand that due to the transformations gltfpack makes some extensions that rely on the original data indexing would invariably be thrashed. However, a lot of metadata could be...

Previously I found very few extensions that can be preserved as is without any custom logic. I'm sure they exist; some KHR material extensions that just add a float or...

> cgltf_load_buffer_base64() doesn't handle this character and instead returns cgltf_result_io_error when it encounters that. This means that you're asking `cgltf_load_buffer_base64` to decode more bytes than are available; the padding should...

> a new compact vertex/index buffer should be generated This should already be the way glTF-Transform works; no change should be required on that front (it looks like it's handled...

Understood. I'm not a lawyer, but none of these seem like blockers to me (pending closer review). meshopt is not a registered trademark.

Some updates! 1. There should not be any copyright/trademark/IP issues as far as I'm concerned. 2. meshoptimizer 0.25 [released](https://github.com/zeux/meshoptimizer/releases/tag/v0.25) last week includes support for color filter and v1 vertex codec...