From 0104bdded0e339989f5a828ef790e6007a63e887 Mon Sep 17 00:00:00 2001 From: Chris Tchou Date: Sat, 28 Nov 2020 11:40:24 -0800 Subject: [PATCH 1/2] Fix for 1292501 --- .../Editor/Material/ShaderGraph/HDShaderPasses.cs | 3 +++ 1 file changed, 3 insertions(+) diff --git a/com.unity.render-pipelines.high-definition/Editor/Material/ShaderGraph/HDShaderPasses.cs b/com.unity.render-pipelines.high-definition/Editor/Material/ShaderGraph/HDShaderPasses.cs index ecf7578f8f0..1889745e1f9 100644 --- a/com.unity.render-pipelines.high-definition/Editor/Material/ShaderGraph/HDShaderPasses.cs +++ b/com.unity.render-pipelines.high-definition/Editor/Material/ShaderGraph/HDShaderPasses.cs @@ -170,6 +170,7 @@ static public PassDescriptor GenerateShadowCaster(bool supportLighting) BlockFields.SurfaceDescription.AlphaClipThreshold, HDBlockFields.SurfaceDescription.AlphaClipThresholdShadow, HDBlockFields.SurfaceDescription.DepthOffset, + HDBlockFields.SurfaceDescription.DiffusionProfileHash // not used, but keeps the UnityPerMaterial cbuffer identical }, // Collections @@ -547,6 +548,7 @@ public static PassDescriptor GenerateTransparentDepthPrepass(bool supportLightin BlockFields.SurfaceDescription.NormalWS, BlockFields.SurfaceDescription.NormalOS, BlockFields.SurfaceDescription.Smoothness, + HDBlockFields.SurfaceDescription.DiffusionProfileHash // not used, but keeps the UnityPerMaterial cbuffer identical } : new BlockFieldDescriptor[] { @@ -554,6 +556,7 @@ public static PassDescriptor GenerateTransparentDepthPrepass(bool supportLightin HDBlockFields.SurfaceDescription.AlphaClipThresholdDepthPrepass, BlockFields.SurfaceDescription.AlphaClipThreshold, HDBlockFields.SurfaceDescription.DepthOffset, + HDBlockFields.SurfaceDescription.DiffusionProfileHash // not used, but keeps the UnityPerMaterial cbuffer identical }, // Collections From 710d90132e2a9914521a9dd7caea4af32353477a Mon Sep 17 00:00:00 2001 From: Chris Tchou Date: Mon, 30 Nov 2020 11:09:49 -0800 Subject: [PATCH 2/2] Adding changelog --- com.unity.render-pipelines.high-definition/CHANGELOG.md | 1 + 1 file changed, 1 insertion(+) diff --git a/com.unity.render-pipelines.high-definition/CHANGELOG.md b/com.unity.render-pipelines.high-definition/CHANGELOG.md index 5fb00e1ab44..fdc7273c1c5 100644 --- a/com.unity.render-pipelines.high-definition/CHANGELOG.md +++ b/com.unity.render-pipelines.high-definition/CHANGELOG.md @@ -30,6 +30,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0. - Fixed an issue where a warning about the static sky not being ready was wrongly displayed. - Fixed the clear coat not being handled properly for SSR and RTR (case 1291654). - Fixed ghosting in RTGI and RTAO when denoising is enabled and the RTHandle size is not equal to the Viewport size (case 1291654). +- Fixed issue where some ShaderGraph generated shaders were not SRP compatible because of UnityPerMaterial cbuffer layout mismatches [1292501] (https://issuetracker.unity3d.com/issues/a2-some-translucent-plus-alphaclipping-shadergraphs-are-not-srp-batcher-compatible) ### Changed - Volume Manager now always tests scene culling masks. This was required to fix hybrid workflow.