Skip to content

Fix sparse indices and values, they are objects not arrays - #9

Open
victor-Lopez25 wants to merge 1 commit into
Pawel82S:masterfrom
victor-Lopez25:master
Open

Fix sparse indices and values, they are objects not arrays#9
victor-Lopez25 wants to merge 1 commit into
Pawel82S:masterfrom
victor-Lopez25:master

Conversation

@victor-Lopez25

Copy link
Copy Markdown
Contributor

I'm slowly doing the glTF 2.0 Khronos tutorial to render glTF and I found sparse accessors were not parsed correctly, so I fixed it. Here are the sources in the glTF 2.0 spec which say the sparse.indices and sparse.values are objects:

and here is the example I found was failing:
https://github.khronos.org/glTF-Tutorials/gltfTutorial/gltfTutorial_005_BuffersBufferViewsAccessors.html#sparse-accessors

I haven't gotten to rendering it yet though 😅 that might be a bit harder.

@victor-Lopez25

Copy link
Copy Markdown
Contributor Author

There might be a similar fix needed in glb parsing but I haven't checked.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant