search

gltf-rs / gltf

A crate for loading glTF 2.0
Apache License 2.0
535 stars 124 forks source link

added cesium rtc #304

Closed bkuster closed 2 years ago

bkuster commented 3 years ago

The Issue

The loader does not support the CESIUM_RTC extension.

Solution

Issues

Im still unsure of what to do with the MODELVIEW semantic (apparenty the extension is supported by gltf 2.0, but they never adapted the spec), so I opted to simply ignore it. Not quite sure what happened to the 1.0 techniques and parameters anyway :shrug:.

IcanDivideBy0 commented 2 years ago

Hello, this crate explicitly states it's for glTF 2.0. As far as I understand, CESIUM_RTC is a glTF 1.0 extension, so i'm not sure this should be added. @aloucks any thought on this ? edit: oh nevermind, was a bit fast on this, even tho it is supported according to the linked issue, i don't see how to integrate the MODELVIEW semantic either :/

bkuster commented 2 years ago

So, this is fairly stale and I guess we will never get the MODELVIEW in here. Since B3DM now provides a batch table semantic RTC_CENTER (see here) I think we can close this.