AI created changes, should it create PR's or work on my own fork

It is actual a limitation, that opens up only for the InstanceData type, for no actual reason. So all sub systems of the engine already support this, but there are a few checks in the VertexBuffer class that limits the use on all other types.

GitHub

Comparing...

A complete 3-D game development suite written in Java. - Comparing jMonkeyEngine:master...zzuegg:fix/vertexbuffer-multi-slot-component-limit · jMonkeyEngine/jmonkeyengine

The changes in the actual engine code are 16 lines in one file. The rest comes from added tests in this example.

I guess the line what is a bug and what is changed behaviour is not that clear. In this example it forces me to split my instance data over multiple vec4 components where an additional mat4 would be enough. Ultimately, i think for no reason since mat4 is already supported and used by InstanceData