search

CesiumGS / cesium-native

Apache License 2.0
402 stars 205 forks source link

Various fixes to correctness of glTF generation. #853

Closed kring closed 4 months ago

kring commented 4 months ago

This is a PR into #842 so merge that first.

There are quite a few small fixes here. See the changelog for the full list.

kring commented 4 months ago

Seems like there were many correctness errors I failed to catch when writing some of this code

Very minor things, really. It's always going to be hard to catch this sort of thing without having a proper glTF validator that we can use from the tests.

Thanks for all the fixes in this PR, they look good!