Skip to content
This repository has been archived by the owner on Dec 22, 2023. It is now read-only.

According to the specs, TriangleWithoutIndices should not render anything #240

Closed
Raveler opened this issue Dec 4, 2019 · 3 comments · Fixed by #242
Closed

According to the specs, TriangleWithoutIndices should not render anything #240

Raveler opened this issue Dec 4, 2019 · 3 comments · Fixed by #242

Comments

@Raveler
Copy link

Raveler commented Dec 4, 2019

The specs say this about a gltf-file without a scene-field specified:

When scene is undefined, runtime is not required to render anything at load time.
Implementation Note: This allows applications to use glTF assets as libraries of individual entities such as materials or meshes.

Since TriangleWithoutIndices does not specify a scene, the default rendered view should be empty, right?

@emackey
Copy link
Member

emackey commented Dec 4, 2019

I read that "not required to render" as being also not prevented from rendering. It would be permissible to show a blank screen, but it would be equally permissible to treat the file's first scene as the default scene automatically.

The scene list is an area of the spec that could use some clarification. For example: KhronosGroup/glTF#1542 and KhronosGroup/glTF#1547

For the sample repo, perhaps we should just fix the sample model to name the correct scene?

@javagl
Copy link
Contributor

javagl commented Dec 4, 2019

This is related to (or basically a duplicate of) #166 - OK, that was my responsibility. I'll re-insert this into my TODO list with higher priority (and leave this issue open until it's really done). Sorry about that.

@Raveler
Copy link
Author

Raveler commented Dec 5, 2019

Alright, thanks! I know it's total nitpicking but those simple models are all used to pick up very detailed flaws in the implementation, so people implementing glTF might think this is such a corner case being tested :).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants