Closed asuessenbach closed 1 month ago
You probably want to mention the issue number somewhere here to tie this back to it: https://github.com/KhronosGroup/Vulkan-Samples/issues/1116
@asuessenbach I've messaged you privately about the cause of the remaining issue. It would also be really good, since these are framework changes, if you could run batch mode yourself and check the box that prompts you to do so.
It would also be really good, since these are framework changes, if you could run batch mode yourself and check the box that prompts you to do so.
Totally agree. I did run the batch 'til its first failure (which was not related to my changes). Plus some of the other samples, but not all. We should strive to resolve all the failures on every sample!
I've messaged you privately about the cause of the remaining issue.
Thanks a lot for spotting the difference! Will have a look at it right now.
Totally agree. I did run the batch 'til its first failure (which was not related to my changes).
The only thing that was preventing a full batch mode running to completion for me was the bug that this change is addressing. It runs fully for me now, so I'm not sure what other things are broken.
It runs fully for me now, so I'm not sure what other things are broken.
I get an obscure error with hello_triangle (loader_add_layer_properties: 'layers' tag not supported until file version 1.0.1, but C:\Windows\System32\DriverStore\FileRepository\nvdmwi.inf_amd64_5542d7b83d2efdd5\nv-vk64.json is reporting version 1
) and a to be fixed error in [hpp_]mesh_shading.
Description
Fixes three errors I introduced with #1091:
Build tested on Win10 with VS2022. Run tested on Win10 with NVidia GPU.
Resolves #1116.
General Checklist:
Please ensure the following points are checked:
[x] This PR describes the scope and expected impact of the changes I am making
Note: The Samples CI runs a number of checks including:
If this PR contains framework changes:
batch
command line argument to make sure all samples still work properlySample Checklist
If your PR contains a new or modified sample, these further checks must be carried out in addition to the General Checklist: