Using the same .tscn with the same .mesh multiple times in the instancer combines them into only one foliage object.

[nan0m]

Terrain3D version

0.9.3a

System information

Godot v4.3.stable.mono - Windows 10.0.19045 - Vulkan (Forward+) - dedicated NVIDIA GeForce RTX 3070 Laptop GPU (NVIDIA; 32.0.15.6636) - 11th Gen Intel(R) Core(TM) i7-11800H @ 2.30GHz (16 Threads)

Is the issue reproducable in the demo?

Not applicable

Issue description

My use case:
I have three differently colored types of bushes all using the same mesh.
Therefore, I load the .tscn that contains the mesh 3x into the foliage instancer each time with a differently colored material.

However, in the end, all three of these foliage slots show the same object with the same material. Painting in the world also paints only one of the three variations.

Workaround:
duplicate the bush .tscn and give each foliage slot a different .tscn to refer to, even when they use the same mesh.

Logs

No response

[TokisanGames]

Godot MMIs do not have material overrides. They have a mesh reference, which has a material.

When you apply a mesh scene, we extract a reference to the mesh from the first MeshInstance3D within the scene. We take the material on the mesh, or the material override in the scene, or our material override, apply it to the mesh and insert it into the MMI.

We don't duplicate the mesh resource. So if you insert the same mesh resource the MMIs are going to be linked to the same mesh resource and the material overwritten on subsequent instances. We could look at duplicating the mesh.

[nan0m]

I wouldn't make it a high priority. Since the workaround is so simple.
Are there plans to move from .tscns to just assigning mesh resources directly, however?

Anw. Also wanted to say that this actually works really well and I am quite happy about it!

[TokisanGames]

Are there plans to move from .tscns to just assigning mesh resources directly, however?

We take in packed scenes, not just tscn. No plans to change that.

[TokisanGames]

This will be fixed by #604, duplicating the provided mesh using the GeometryInstance override slot so different materials can be used for the same PackedScene.