Ecosyste.ms: Timeline

Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.

arno-lunarg

arno-lunarg created a review on a pull request on KhronosGroup/Vulkan-ValidationLayers
I think there are needless deep vector copies, aside that LGTM

View on GitHub

arno-lunarg closed a pull request on KhronosGroup/Vulkan-ValidationLayers
GPU-AV: Add debug label regions to instrumentation error messages
Still need to add it to debug printf
arno-lunarg created a review comment on a pull request on KhronosGroup/Vulkan-ValidationLayers
aaaand did that

View on GitHub

arno-lunarg created a review on a pull request on KhronosGroup/Vulkan-ValidationLayers

View on GitHub

arno-lunarg created a comment on a pull request on KhronosGroup/Vulkan-ValidationLayers
Will add support for debug printf and tests with secondary command buffers tomorrow

View on GitHub

arno-lunarg pushed 10 commits to arno-gpuav-debug-labels arno-lunarg/Vulkan-ValidationLayers
  • layers: Fix message formatting in WSI validation dda0b17
  • layers: Fix handling of deserialized accel struct After a deserialization, VVL has no info on how the deserialized a... d952704
  • tests: Deserialized accel struct no false positive 21e6c02
  • layers: Fix VU 04087 Need to check that family indices are not equal 3a6023c
  • tests: Add ownership transfer tests for 09049 and 09052 There is validation but not the tests for: VUID-VkBufferMemo... 8bbdd7e
  • layers: Improve ValidateDescriptor aff9d28
  • gpu: Prevent false positives for aliased bindings 62286d3
  • tests: Prevent false positives for aliased bindings e7100bb
  • gpuav: Add debug label regions to instrumentation errors 7bae578
  • tests: Add GPU-AV debug label regions tests de4b2ec

View on GitHub

arno-lunarg created a review comment on a pull request on KhronosGroup/Vulkan-ValidationLayers
But I can just move `LabelCommand` outside `vvl::CommandBuffer`

View on GitHub

arno-lunarg created a review on a pull request on KhronosGroup/Vulkan-ValidationLayers

View on GitHub

arno-lunarg created a review comment on a pull request on KhronosGroup/Vulkan-ValidationLayers
[For the compiler to know that `LabelCommand` is within ](https://stackoverflow.com/questions/1021793/how-do-i-forward-declare-an-inner-class)

View on GitHub

arno-lunarg created a review on a pull request on KhronosGroup/Vulkan-ValidationLayers

View on GitHub

arno-lunarg created a review comment on a pull request on KhronosGroup/Vulkan-ValidationLayers
ah leftover!

View on GitHub

arno-lunarg created a review on a pull request on KhronosGroup/Vulkan-ValidationLayers

View on GitHub

arno-lunarg created a review comment on a pull request on KhronosGroup/Vulkan-ValidationLayers
I had that lol, but then I was like "what I really want is for some string to not be in one particular message, not necessarily all of them"

View on GitHub

arno-lunarg created a review on a pull request on KhronosGroup/Vulkan-ValidationLayers

View on GitHub

arno-lunarg created a review comment on a pull request on KhronosGroup/Vulkan-ValidationLayers
I have to use reverse iterators, doing a reverse loop by hand is messy, it looks ass but at least I am sure the reverse loop is not messed up : /

View on GitHub

arno-lunarg created a review on a pull request on KhronosGroup/Vulkan-ValidationLayers

View on GitHub

arno-lunarg created a review comment on a pull request on KhronosGroup/Vulkan-ValidationLayers
I will remove `label_command`, but keep the others

View on GitHub

arno-lunarg created a review on a pull request on KhronosGroup/Vulkan-ValidationLayers

View on GitHub

arno-lunarg created a review comment on a pull request on KhronosGroup/Vulkan-ValidationLayers
We have to carry a value in the logging lambda, a pointer/reference would be dangling at error logging time. And that value may not be there if not within a debug label region, so have to use an op...

View on GitHub

arno-lunarg created a review on a pull request on KhronosGroup/Vulkan-ValidationLayers

View on GitHub

arno-lunarg created a review comment on a pull request on KhronosGroup/Vulkan-ValidationLayers
no because `LabelCommand` is withing `vvl::CommandBuffer` : /

View on GitHub

arno-lunarg created a review on a pull request on KhronosGroup/Vulkan-ValidationLayers

View on GitHub

arno-lunarg opened a pull request on KhronosGroup/Vulkan-ValidationLayers
GPU-AV: Add debug label regions to instrumentation error messages
Still need to add it to debug printf
arno-lunarg created a branch on arno-lunarg/Vulkan-ValidationLayers

arno-gpuav-debug-labels - Vulkan Validation Layers

arno-lunarg created a review comment on a pull request on KhronosGroup/Vulkan-ValidationLayers
Not sure why this `dummy` assignment is not considered dead code, and the following `+=` is, I trust your judgment on this!

View on GitHub

arno-lunarg created a review on a pull request on KhronosGroup/Vulkan-ValidationLayers

View on GitHub

arno-lunarg deleted a branch arno-lunarg/Vulkan-ValidationLayers

arno-fix-deserialized-accel-structs-validation

arno-lunarg pushed 2 commits to main KhronosGroup/Vulkan-ValidationLayers
  • layers: Fix handling of deserialized accel struct After a deserialization, VVL has no info on how the deserialized a... d952704
  • tests: Deserialized accel struct no false positive 21e6c02

View on GitHub

arno-lunarg closed an issue on KhronosGroup/Vulkan-ValidationLayers
Apparent "must have the same value" false-positives when `UPDATE`ing (de)serialized Acceleration Structures
**Environment:** - OS: Linux/Windows - GPU and driver version: Intel Arc, amdgpu, ... - SDK or header version if building from repo: 1.3.290 - Options enabled (synchronization, best practic...
arno-lunarg closed a pull request on KhronosGroup/Vulkan-ValidationLayers
Fix state tracking and validation of deserialized acceleration structures
Closes https://github.com/KhronosGroup/Vulkan-ValidationLayers/issues/8590
Load more