Neui (00eafc6d) at 27 Oct 14:13

Ok I added a test now and verified that it fails on master but passes with this fix. Not sure if I am testing "too much" for a unit test and should just have called the actual conversion function by itself.

(Also from what I understand it doesn't "deduplicate" BufferViews that share the same Buffer, like for interleaved stuff, but this might be stuff for a later MR)

Neui (b8ad30e1) at 08 Feb 23:10

Previously, it tries to compensate for alignment but it does it too early. Going "past the buffer" in python just truncates the resulting array, so no padding bytes are actually added to the output, essentially causing a desync between the buffer contents and recorded offsets, so GLB consumers see nonsensical data.

This fixes it by actually adding zero bytes for padding to the output.

Neui (c3ab9724) at 08 Feb 15:31

Neui (ea272c7a) at 08 Feb 15:25