We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
According to the specification, the glTF.scene field is optional (i.e "not required").
But there are at least two places where gltf.scenes[gltf.scene] seems to be accessed directly:
gltf.scenes[gltf.scene]
(others may exist - I did not systematically search for all possible occurrences).
When trying to load a glTF without a scene property, this obviously leads to the error
scene
TypeError: t.scenes[t.scene] is undefined
(This basically applies to all the tutorial models. Right now, it can be tested quickly at https://cx20.github.io/gltf-test/examples/cesium/index.html?category=tutorialModels&model=SimpleTexture&scale=1&type=glTF-Embedded-buffer )
(BTW: The linked lines also raise the question of whether cesium would be capable of handling multiple scenes, but this may be another issue...)
The text was updated successfully, but these errors were encountered:
CC KhronosGroup/glTF#815
Sorry, something went wrong.
Seems to be resolved 👴
No branches or pull requests
According to the specification, the glTF.scene field is optional (i.e "not required").
But there are at least two places where
gltf.scenes[gltf.scene]
seems to be accessed directly:(others may exist - I did not systematically search for all possible occurrences).
When trying to load a glTF without a
scene
property, this obviously leads to the error(This basically applies to all the tutorial models. Right now, it can be tested quickly at https://cx20.github.io/gltf-test/examples/cesium/index.html?category=tutorialModels&model=SimpleTexture&scale=1&type=glTF-Embedded-buffer )
(BTW: The linked lines also raise the question of whether cesium would be capable of handling multiple scenes, but this may be another issue...)
The text was updated successfully, but these errors were encountered: