Skip to content

Conversation

remi-chapelain
Copy link
Contributor

@remi-chapelain remi-chapelain commented Apr 30, 2021

Purpose of this PR

Following this issue, decided to add coverage for debug mode to avoid future regressions about this. In that objective, 8 tests have been added to cover all raytraced effects debug mode. Those tests are just duplicate of already existsing test testing specific ray-traced effects.

  • 2000_Debug_ScreenSpaceGlobalIllumination
  • 2001_Debug_ScreenSpaceReflections
  • 2002_Debug_ScreenSpaceReflectionsOnTransparents
  • 2003_Debug_RecursiveRendering
  • 2004_Debug_ContactShadows
  • 2005_Debug_ScreenSpaceAmbientOcclusion
  • 2006_Debug_ScreenSpaceShadows
  • 2007_Debug_LightCluster

The only tests that's missing right now would be RTSSSurface but I postponed it because it might be broken right now.


Testing status


Comments to reviewers

/!\ this needs to be merged after this commit hits master or retargeted to hd/bugfix otherwise the light cluster test will fail.

@remi-chapelain
Copy link
Contributor Author

remi-chapelain commented Apr 30, 2021

2000_Debug_ScreenSpaceGlobalIllumination
2000_Debug_ScreenSpaceGlobalIllumination

2001_Debug_ScreenSpaceReflections
2001_Debug_ScreenSpaceReflections

2002_Debug_ScreenSpaceReflectionsOnTransparents
2002_Debug_ScreenSpaceReflectionsOnTransparents

2003_Debug_RecursiveRendering
2003_Debug_RecursiveRendering

2004_Debug_ContactShadows (for some reasons contacts shadows reference image has alpha of 0 but still has proper data on RBG channels)
2004_Debug_ContactShadows

image

2005_Debug_ScreenSpaceAmbientOcclusion
2005_Debug_ScreenSpaceAmbientOcclusion

2006_Debug_ScreenSpaceShadows
2006_Debug_ScreenSpaceShadows

2007_Debug_LightCluster
2007_Debug_LightCluster

@remi-chapelain remi-chapelain requested review from a team and sebastienlagarde April 30, 2021 10:03
@sebastienlagarde
Copy link
Contributor

awesome, thanks for this PR

@sebastienlagarde sebastienlagarde marked this pull request as ready for review April 30, 2021 11:13
@sebastienlagarde sebastienlagarde merged commit 1f447eb into master Apr 30, 2021
@sebastienlagarde sebastienlagarde deleted the hd/add-full-screen-debug-mode-coverage branch April 30, 2021 16:57
sebastienlagarde pushed a commit that referenced this pull request May 1, 2021
…ests Projects (#4391)

* 5 new lighting debug mode scene

* RTAO + SSShadows

* build settings

* Light Cluster scene

* Contact shadows correct captures using BackBuffer

* ContactShadow scene

* forgot meta files
sebastienlagarde pushed a commit that referenced this pull request May 2, 2021
…ests Projects (#4391)

* 5 new lighting debug mode scene

* RTAO + SSShadows

* build settings

* Light Cluster scene

* Contact shadows correct captures using BackBuffer

* ContactShadow scene

* forgot meta files
sebastienlagarde added a commit that referenced this pull request May 5, 2021
* [HDRP][DXR] Add coverage for lighting full screen debug mode in DXR Tests Projects (#4391)

* 5 new lighting debug mode scene

* RTAO + SSShadows

* build settings

* Light Cluster scene

* Contact shadows correct captures using BackBuffer

* ContactShadow scene

* forgot meta files

* Add coverage for path tracer alpha channel support (#4393)

* add-pt-alpha-automated-test

* Tidy up 5001 scene

* Disable DXR XR test (#4403)

* Add support for lighting full screen debug mode for HDRP tests #4367

* Fix automated tests

* upgrade path tracing alpha test

* 5001 updated unreadable image

* disable DXR XR 2005

Co-authored-by: Rémi Chapelain <57442369+remi-chapelain@users.noreply.github.com>
Co-authored-by: Remi Chapelain <remi.chapelain@unity3d.com>
sebastienlagarde added a commit that referenced this pull request May 29, 2021
* [HDRP][DXR] Add coverage for lighting full screen debug mode in DXR Tests Projects (#4391)

* 5 new lighting debug mode scene

* RTAO + SSShadows

* build settings

* Light Cluster scene

* Contact shadows correct captures using BackBuffer

* ContactShadow scene

* forgot meta files

* Added information about ray tracing light culling (#4392)

* Added ray tracing light culling section and moved a couple of things around

* Update Ray-Tracing-Getting-Started.md

Co-authored-by: sebastienlagarde <sebastien@unity3d.com>

* Added more pages to help users with runtime usage #4394

* Disable DXR XR test (#4403)

* Add support for lighting full screen debug mode for HDRP tests #4367

* Update CHANGELOG.md

* Fix automated tests

* upgrade path tracing alpha test

* 5001 updated unreadable image

* update failing test scene

* Update failing scenes

* Changing geometry in 708/711 tests

It was not appearing for some reasons

* Tentative fix

* Debug 'debug test scene'

* Debug

* new DebugDisplaySettings

* revert static variable to private

* Disable debug test scenes and reverting changes

Co-authored-by: Rémi Chapelain <57442369+remi-chapelain@users.noreply.github.com>
Co-authored-by: Lewis Jordan <lewis.jordan@hotmail.co.uk>
Co-authored-by: Remi Chapelain <remi.chapelain@unity3d.com>
pastasfuture pushed a commit that referenced this pull request Aug 20, 2021
* [HDRP][DXR] Add coverage for lighting full screen debug mode in DXR Tests Projects (#4391)

* 5 new lighting debug mode scene

* RTAO + SSShadows

* build settings

* Light Cluster scene

* Contact shadows correct captures using BackBuffer

* ContactShadow scene

* forgot meta files

* Added information about ray tracing light culling (#4392)

* Added ray tracing light culling section and moved a couple of things around

* Update Ray-Tracing-Getting-Started.md

Co-authored-by: sebastienlagarde <sebastien@unity3d.com>

* Added more pages to help users with runtime usage #4394

* Disable DXR XR test (#4403)

* Add support for lighting full screen debug mode for HDRP tests #4367

* Update CHANGELOG.md

* Fix automated tests

* upgrade path tracing alpha test

* 5001 updated unreadable image

* update failing test scene

* Update failing scenes

* Changing geometry in 708/711 tests

It was not appearing for some reasons

* Tentative fix

* Debug 'debug test scene'

* Debug

* new DebugDisplaySettings

* revert static variable to private

* Disable debug test scenes and reverting changes

Co-authored-by: Rémi Chapelain <57442369+remi-chapelain@users.noreply.github.com>
Co-authored-by: Lewis Jordan <lewis.jordan@hotmail.co.uk>
Co-authored-by: Remi Chapelain <remi.chapelain@unity3d.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants