Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add D3D12 test failed callback for pixel history #3112

Merged
merged 1 commit into from
Nov 10, 2023

Conversation

skarolewics
Copy link
Contributor

This callback determines which tests passed or failed for a given draw during pixel history. The set of tests mirrors the Vulkan implementation, with some adjustments for D3D12 capabilities. The code for updating PixelModification structs during pixel history is included as well since they're related.

@Zorro666 Zorro666 self-requested a review November 6, 2023 09:36
Copy link
Collaborator

@Zorro666 Zorro666 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you - all looks very good.

A couple of very minor queries

renderdoc/driver/d3d12/d3d12_pixelhistory.cpp Outdated Show resolved Hide resolved
renderdoc/driver/d3d12/d3d12_pixelhistory.cpp Outdated Show resolved Hide resolved
renderdoc/driver/d3d12/d3d12_pixelhistory.cpp Show resolved Hide resolved
This callback determines which tests passed or failed for a given
draw during pixel history. The set of tests mirrors the Vulkan
implementation, with some adjustments for D3D12 capabilities.
The code for updating PixelModification structs during pixel
history is included as well since they're related.
@Zorro666 Zorro666 merged commit 73c604d into baldurk:v1.x Nov 10, 2023
16 checks passed
@Zorro666
Copy link
Collaborator

Thank you again for the code/feature and making small tweaks during the landing process

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants