[Potential Bug] Render Complete Semaphore is specified using `currentFrame` rather than `imageIndex` which can cause sync validation errors.

[Mjrlun]

As title suggests. I noticed this after comparing my work with yours (having previously followed the tutorial before), and having conferred with someone in the Vulkan discord.

The solution would be to create an array of Semaphores and access them using the imageIndex passed back from acquireNextImage, in place of the current renderCompleteSemaphore within the SyncSemaphores record.
You could also do the same by specially accessing the SyncSemaphores array for the render complete semaphore: syncSemaphores[imageIndex].renderCompleteSemaphore().vkSemaphore

[Mjrlun]

Additionally, the index for the LightingFrameBuffer VkFrameBuffers should also be using this same imageIndex, rather than how it currently uses the current frame.

[lwjglgamedev]

Ok,

I saw this issue some weeks ago, becuase it caused validation warnings. Perviously, everything as reported as correct. I was not using the proper image index returned while acquiring the image. But I already uploaded a fix for that, and the valiaton error messages are goone. Therfore, it seems to be working now.

In any case, just for you to know. I'm currently working on a new version of the book with the following deisgn goals:

• Use Vulkan 1.3.
• Use dynamyc render (no more render passes). I feel this is the proper way, it is more explicit sometines, but no burden on render passes, frame buffers, and implicit barriers and transitions though dependencies.
• Better overall handling, in terms of descriptor sets, and better model for swap chain image acquisition.

I'm also planning in including a sample for PhysX.

It is taking some time, but I'm working on it.

[Mjrlun]

Sounds awesome! I will take a look at it when it's done, although not looking forward to more sync error nightmares. Hopefully it will run faster though :)

[lwjglgamedev]

I can give access to the working source code if you want to have a look. I'm regulary testing with sync validation layers on.

[Mjrlun]

Sure if you want. Just DM me on discord I guess.