zeux / niagara

A Vulkan renderer written from scratch on stream
MIT License
1.26k stars 72 forks source link

More synchronization optimizations #24

Closed YunHsiao closed 2 years ago

YunHsiao commented 2 years ago

Hi Arseny,

Glad to see the new niagara stream (Your streams are literally my starting point to learn Vulkan back then!), sorry about not fully fixing the problem last time (just touched where the SyncVal then has taken me). I just couldn't stop smiling knowing that you too are struggling with it a bit ;P

So after watching the playback, this time I took a little closer look and found some further improvements over the usages on top of the latest update, and everything still works after the change with no validation error on all paths. Let me know if it helps :)

zeux commented 2 years ago

Thanks!