Skip to content
New issue

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

GLTF with 3 UV Maps throws an error #12995

Open
ethereumdegen opened this issue Apr 16, 2024 · 1 comment
Open

GLTF with 3 UV Maps throws an error #12995

ethereumdegen opened this issue Apr 16, 2024 · 1 comment
Labels
A-Assets Load files from disk to use for things like images, models, and sounds A-Diagnostics Logging, crash handling, error reporting and performance analysis C-Bug An unexpected or incorrect behavior C-Usability A simple quality-of-life change that makes Bevy easier to use

Comments

@ethereumdegen
Copy link
Contributor

Bevy version

Release 0.13.2

What you did

Try to load a GLTF with 3 uv maps like this (inside of gltf file) :

"meshes":[
		{
			"name":"Mesh.001",
			"primitives":[
				{
					"attributes":{
						"POSITION":0,
						"NORMAL":1,
						"TEXCOORD_0":2,
						"TEXCOORD_1":3,
						"TEXCOORD_2":4
					},
					"indices":5,
					"material":0
				},
				{
					"attributes":{
						"POSITION":6,
						"NORMAL":7,
						"TEXCOORD_0":8,
						"TEXCOORD_1":9,
						"TEXCOORD_2":10
					},
					"indices":11,
					"material":1
				}
			]
		}
	],

What went wrong

Bevy throws an error

WARN bevy_gltf::loader: Unknown vertex attribute TEXCOORD_2

Additional information

If this is supposed to error, explain it in the docs and also when the error is thrown, print out the path or name of the GLTF that is failing to load . If this is not supposed to error and that functionality should be supported, support it in the vertex attributes loader for gltf.

@ethereumdegen ethereumdegen added C-Bug An unexpected or incorrect behavior S-Needs-Triage This issue needs to be labelled labels Apr 16, 2024
@NthTensor NthTensor added A-Assets Load files from disk to use for things like images, models, and sounds and removed S-Needs-Triage This issue needs to be labelled labels Apr 21, 2024
@NthTensor
Copy link
Contributor

Looks like we are not fully compliant with the gltf spec. See the definition of TEXCOORD_n in the meshes chapter.

Not sure if we want to/plan to add support for multiple UVs. If we don't we should fix this error to be more explanatory.

@NthTensor NthTensor added C-Usability A simple quality-of-life change that makes Bevy easier to use A-Diagnostics Logging, crash handling, error reporting and performance analysis labels Apr 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Assets Load files from disk to use for things like images, models, and sounds A-Diagnostics Logging, crash handling, error reporting and performance analysis C-Bug An unexpected or incorrect behavior C-Usability A simple quality-of-life change that makes Bevy easier to use
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants