In EXT_structural_metadata, the arrayOffsets of a PropertyTableProperty represent array indices. However, Cesium Native treats them as byte offsets that are multiples of sizeof(T), which is wrong. This PR corrects this behavior and edits the unit tests for the proper values.
Also, I arbitrarily found and replaced instances of glm::i32mat2x2 -> glm::imat2x2, so the diff looks bigger than it actually needs to be 😅
kring
commented
7 months ago
Depends on #794, so merge that first.
In
EXT_structural_metadata, thearrayOffsetsof aPropertyTablePropertyrepresent array indices. However, Cesium Native treats them as byte offsets that are multiples ofsizeof(T), which is wrong. This PR corrects this behavior and edits the unit tests for the proper values.Also, I arbitrarily found and replaced instances of
glm::i32mat2x2 -> glm::imat2x2, so the diff looks bigger than it actually needs to be 😅