Add uv transforms to thininstances

[unicomp21]

Add uv transforms to thininstances.

[deltakosh]

Please reference the forum post

[unicomp21]

https://forum.babylonjs.com/t/thin-instances-uv-attribute-weird-result/34061/14?u=unicomp21

[sebavan]

I am wondering if it should be supported by default or wether a material plugin only ? This looks like a pretty edge case feature and I am not sure we should embed it by default ?

[unicomp21]

I think it could be edge ... until it's not. The potential here for a non-repeating (thanks to variation in texture), volume filling primitive, is pretty significant.

[sebavan]

By edge case I mean not everybody will use it and thus I d prefer it to be on demand to not increase the size of the code in most of the experiences.

[unicomp21]

How much of a size increase will there be?

[sebavan]

not sure but we could say the same for any small features resulting in overall big additions :-)

I am not saying we should not add the feature, but I d rather keep it as a material plugin that people need to import in order to benefit from it instead of embedding it in the material.

[github-actions[bot]]

This issue has been automatically staled because it has been inactive for more than 14 days. Please update to "unstale".

[thomlucc]

https://forum.babylonjs.com/t/add-uv-transforms-to-thininstances/52140