Fix: Set queue priority to 1.0f in all samples and docs#166
Fix: Set queue priority to 1.0f in all samples and docs#166marty-johnson59 merged 1 commit intoKhronosGroup:mainfrom
Conversation
Where is that coming from? Do you have sources for this? |
The use of The Vulkan specification (section 5.3.4 – Queue Priority) defines |
I have never heard of that nor did I ever experience that. What is that "deprioritization" part based on? |
It's not based on a known issue — just on the spec wording that higher-priority queues may get more time. I used “deprioritization” in the sense of avoiding future ambiguity if other queues were added. It’s mostly about explicit intent and clearer code, especially in tutorials. |
|
We agreed on using 0.5f instead for the samples and will change those. Can you update your PR accordingly? |
654bb6e to
e936103
Compare
Updated to 0.5f as requested. Thanks for the feedback! |
|
Looks good to me. We just merged a larger PR that reformats code, so in order to merge this you need to fix the merge conflicts. |
6b4e7dc to
994c895
Compare
Replaced all instances of:
with:
Updated in both
.adocand.cppfiles (33 total), to reflect best practice.1.0fis the recommended queue priority, even in single-queue setups, to avoid confusion for beginners and ensure optimal execution priority.