Skip to content

Allow ShaderGraph generation to generate multiple render state [Skip CI] #1039

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

Merged
merged 27 commits into from
Jul 2, 2020

Conversation

alelievr
Copy link
Member

@alelievr alelievr commented Jun 29, 2020

Purpose of this PR

Allow to generate multiple render state of the same type, needed for some use cases like HDRP unlit shader:
image


Testing status

HDRP_Tests project is green locally. I tested the generation of the HDRP Unlit shader and it is now correct.
https://yamato.prd.cds.internal.unity3d.com/jobs/902-Graphics/tree/sg%252Ffix%252Fmultiple-renderstate/.yamato%252F_abv.yml%2523all_project_ci_fast-trunk/2662083/job/pipeline

JulienIgnace-Unity and others added 5 commits June 29, 2020 14:29
* Implemented Probe Volume light list build.

* Implement transparency overdraw.

* Implement XR occlusion mesh

* Fix when no movecs are available.

* Remove unnecessary workaround.

* Fixed AO texture binding and usage.

* Implement transparent SSR and fixed transparent depth prepass

* Fixed a leak with the resource purge system.

Fixed a gcalloc
Properly bind normal and movec textures to SSR.

* Temporary fix to a texture binding error with deferred compute shaders.

* Fixed null ref

* Fixed area shadows

* Enabled test 2501 LightLayers

* Enabled SSAO tests 2601/2602

* Fixed debug mode background color

Re-enabled test 3002 DebugLightLayers

* Fixed GCAlloc in PushFullScreenDebug

Re-enabled test 4013 TransparentMotionVector

* Fixed GCAlloc in Resolve MSAA functions

Re-enabled test 5009 HDRISkyFlow

* Reenabled 8106

* Fixed MSAA prepass and camera motion vector

Re-enabled tests 9401

* Re-enabled tests 9402

* Re-enabled 9500, 9603 and 9604

* Fixed object motion vector pass with MSAA.

Re-enabled tests 9601 and 9602

* Fixed AOV output

Re-enabled test 9901

* Fixed binding of depth texture to volumetric lighting shader.

* Custom Passes implementation WIP

* Remove comment

* Re-enabled tests 9700 9701 for custom passes.

* Add comment.

* typo

* Fixed a null ref and binding of color pyramid.

* Implemented MSAA resolve for custom pass.
@alelievr alelievr requested a review from a team as a code owner June 29, 2020 18:02
@alelievr alelievr self-assigned this Jun 29, 2020
@sebastienlagarde sebastienlagarde changed the base branch from HDRP/staging to master June 29, 2020 19:08
@sebastienlagarde sebastienlagarde changed the base branch from master to HDRP/staging June 29, 2020 19:08
anisunity and others added 10 commits June 30, 2020 12:01
…#976)

* Added an example CloudLayer render texture + rotation for cloud layer

* Fix shader warnings

* Better Inspector. Hacky fix for PBR sky

* Add name of default tetxure in doc

* Section for duplicating asset

* Improve doc
* Fix test scene description asset UI

* Added doc
…on in ray tracing effects (1258547). (#1042)

* Added an option to extend the camera culling for skinned mesh animation in ray tracing effects (1258547).

* Improved tooltip

Co-authored-by: Remi Chapelain <remi.chapelain@unity3d.com>
Co-authored-by: sebastienlagarde <sebastien@unity3d.com>
* Change transparent SSR display name

* changelog

* Add frame settings docs update
Co-authored-by: sebastienlagarde <sebastien@unity3d.com>
…re (#1061)

* Added missing tooltips in the physical camera UI

* Fix tooltips for DoF

* Revert doc change

* Custom drawing for the aperture

* Code cleanup

* Add missing comment

* Changelog

* Clamp the aperture to the correct range, avoid using hard-coded constants

* Fixed more tooltips + typo
@alelievr alelievr requested review from a team as code owners July 1, 2020 12:18
@alelievr alelievr changed the base branch from HDRP/staging to master July 1, 2020 12:18
@alelievr alelievr requested review from martint-unity and removed request for a team July 1, 2020 12:18
@alelievr alelievr changed the base branch from master to HDRP/staging July 1, 2020 12:19
@alelievr alelievr removed request for a team and martint-unity July 1, 2020 12:19
alelievr and others added 3 commits July 1, 2020 14:22
* Add guards to shadow includes

* changelog

Co-authored-by: sebastienlagarde <sebastien@unity3d.com>
…on null rthandles (#1051)

* Fix exception spam

* Revert "Fix exception spam"

This reverts commit 8d91753.

* Fix exception on null RTHandle at the root of the problem.
Copy link
Contributor

@alindmanUnity alindmanUnity left a comment

Choose a reason for hiding this comment

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

approving SG changes

FrancescoC-unity and others added 8 commits July 1, 2020 18:27
* Force scene repaint on change of overridden exposure

* changelog

Co-authored-by: sebastienlagarde <sebastien@unity3d.com>
…e (1259930). (#1070)

Co-authored-by: sebastienlagarde <sebastien@unity3d.com>
* Fixed the non-editable transparent writes motion vector and back then front rendering

* Fixed alpha to mask reset in the material UI

* Updated changelog

* Fixed show after post process being visible in Lit SG material UI

* Fixed displacement mode not showing in the material UI

* Bump material version on HDRPProjectSettings

* Moved depth offset keyword to base unlit

* Fixed material multi-selection at several places

* Moved common.hlsl and shaderpass.cs.hlsl before define block in shadergraph

* Fixed depth offset displayed in material but disabled in SG

* update HDRP test material

* update DXR material

* Update 709_CameraExtendedCulling.png

Co-authored-by: Antoine Lelièvre <antoinel@unity3d.com>
* Fixed an issue where opening the look dev window with the light theme would make the window blink and eventually crash unity.

* Update changelog.

Co-authored-by: sebastienlagarde <sebastien@unity3d.com>
@sebastienlagarde sebastienlagarde changed the base branch from HDRP/staging to master July 2, 2020 01:48
@sebastienlagarde sebastienlagarde changed the title Allow ShaderGraph generation to generate multiple render state Allow ShaderGraph generation to generate multiple render state [Skip CI] Jul 2, 2020
@sebastienlagarde sebastienlagarde merged commit f8b473d into master Jul 2, 2020
@sebastienlagarde sebastienlagarde deleted the sg/fix/multiple-renderstate branch July 2, 2020 04:00
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.

9 participants