-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Conversation
For AdvancedMaterial, SimpleOpacity and SimpleSkin, the update is sill pending (and indicated by a prominent note in the README). They will be updated ASAP.
@lexaknyazev about when do you think the validator will be ready for us to test these? |
No haste - Alexey is probably busy with more urgent tasks right now. I'll also review the models again ASAP, which goes hand in hand with checking the spec changes. The "big" changes actually affect the models that are not updated yet (materials and skinning), and I'll try to create further test models (morphing, sparse accessors...), so there's still a lot of work to do to consolidate all this anyhow. |
Fixes most of #53, see #53 (comment). |
@javagl Is the position attribute of SimpleMaterial.gltf in this PR correct? /cc @takahirox Relate : mrdoob/three.js#11326 (comment) |
@cx20 That's hard to say. It may not be correct. I think it once was correct. At least, once it was supposed to be correct (in the future) : KhronosGroup/glTF#789 (comment) The |
@javagl
|
Most of the test models have been updated to 2.0. They may have to be reviewed, but in general, they should be compliant to the spec. A validation might reveal details that have to be fixed.
Only the AdvancedMaterial, SimpleOpacity and SimpleSkin model updates are still pending. These updates will depend on the MaterialCommon extension and the latest changes of skinning, which are not integrated yet.