glTF: Use the vertex colors when available

[capnm]

Godot version:

tip de0fb07

Issue description:

After the import you have to change each material manually:

[fire]

Can you post a sample project?

[paulmiller]

I think they're saying they want "use as albedo" to be enabled automatically if the model has vertex color info. I personally wouldn't want that - at least, not for materials outside the model's own gltf file. It'd be very confusing for the material's properties to change depending on which models are using it.

[fire]

I'm not able to find a gltf2 spec description that allows this. https://github.com/KhronosGroup/glTF/tree/master/specification/2.0

Edited:

So I'm asking for a valid gltf2 file.

[capnm]

So I'm asking for a valid gltf2 file.

It feels like someone is calling me from the grave :-)

waiting-for-godot-5.0.zip

Validation report

    Format: glTF 2.0
    Generator: Khronos glTF Blender I/O v1.2.75
    Stats:
        1 draw calls
        0 animations
        0 materials
        22269 vertices
        8217 triangles
    Extensions: None

https://docs.blender.org/manual/en/latest/addons/import_export/scene_gltf2.html#geometry

[fire]

Try this #41007.

[capnm]

Try this #41007.

I guess for 3.2.x this should be:

				Ref<Material> mat;
				mat.instance();
				mat->set_flag(Material::FLAG_ALBEDO_FROM_VERTEX_COLOR, true);

right?

[fire]

Looks good on a first glance. Haven't ported the vertex color patch over though. In the middle of something.

[capnm]

Looks good on a first glance. Haven't ported the vertex color patch over though. In the middle of something.

I figured it out, someone renamed the SpatialMaterial 🤕
Also works with objects that have textured and vertex-colored surfaces combined.

Added to my 3.2.x patch set :-)
Thanks!

texture-and-vertex-color-a01.zip