I am looking at a Blender Metal Backend bug (so in our codebase). The SSBO Vertex Fetch API was introduced by Michael PW to be able to workaround Metal's lack of geometry…
@James-McCarthy-4 My bad. You are right. It is in fact index_base_
(or baseVertex
) that should be added to the index value from the index buffer.
I believe the indexBufferOffset
is…
So the issue is that the new default normal attribute is (0,0,1)
for loose edges, but shader expect default to be (0,0,0)
.
bool no_attr = all(equal(nor, vec3(0)));
@HooglyBoogly…
@James-McCarthy-4 Yes it is indexBufferOffset
but in vertex, not in bytes. See gpu::IndexBuf::index_start_
.
It is not taken into account inside the RESOLVE_VERTEX macro of the vertex fetch…
@James-McCarthy-4 This is the issue I was mentioning in the meeting today. I am not sure how hard is it to fix for 4.2 / 4.3.
But it will definitely be fixed in 4.4 after replacing the Overlay…