[HDRP] Merge HD/bugfix #6387

Author: sebastienlagarde

com.unity.render-pipelines.core/CHANGELOG.md

@@ -10,6 +10,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
 - Fixed potentially conflicting runtime Rendering Debugger UI command by adding an option to disable runtime UI altogether (1345783).
 - Fixed issue when changing volume profiles at runtime with a script (case 1364256).
 - Fixed XR support in CoreUtils.DrawFullscreen function.
+- Fixed an issue causing Render Graph execution errors after a random amount of time.
 
 ## [10.7.0] - 2021-07-02
 

com.unity.render-pipelines.core/Editor/Debugging/DebugWindow.cs

@@ -378,7 +378,12 @@ void OnGUI()
             //GUILayout.Button(k_LoadButtonContent, EditorStyles.toolbarButton);
             //GUILayout.Button(k_SaveButtonContent, EditorStyles.toolbarButton);
             if (GUILayout.Button(k_ResetButtonContent, EditorStyles.toolbarButton))
+			{
                 DebugManager.instance.Reset();
+                DestroyWidgetStates();
+                UpdateWidgetStates();
+            }
+
             GUILayout.EndHorizontal();
 
             using (new EditorGUILayout.HorizontalScope())

com.unity.render-pipelines.core/Editor/Volume/VolumeEditor.cs

@@ -45,7 +45,7 @@ void RefreshEffectListEditor(VolumeProfile asset)
         {
             m_ComponentList.Clear();
 
-            asset.Sanitize();
+            asset?.Sanitize();
 
             if (asset != null)
                 m_ComponentList.Init(asset, new SerializedObject(asset));

com.unity.render-pipelines.core/Runtime/RenderGraph/RenderGraphResources.cs

@@ -46,14 +46,21 @@ public bool IsValid()
 
         static public void NewFrame(int executionIndex)
         {
+            uint previousValidBit = s_CurrentValidBit;
             // Scramble frame count to avoid collision when wrapping around.
             s_CurrentValidBit = (uint)(((executionIndex >> 16) ^ (executionIndex & 0xffff) * 58546883) << 16);
             // In case the current valid bit is 0, even though perfectly valid, 0 represents an invalid handle, hence we'll
             // trigger an invalid state incorrectly. To account for this, we actually skip 0 as a viable s_CurrentValidBit and
             // start from 1 again.
-            if (s_CurrentValidBit == 0)
+            // In the same spirit, s_SharedResourceValidBit is reserved for shared textures so we should never use it otherwise
+            // resources could be considered valid at frame N+1 (because shared) even though they aren't.
+            if (s_CurrentValidBit == 0 || s_CurrentValidBit == s_SharedResourceValidBit)
             {
-                s_CurrentValidBit = 1 << 16;
+                // We need to make sure we don't pick the same value twice.
+                uint value = 1;
+                while (previousValidBit == (value << 16))
+                    value++;
+                s_CurrentValidBit = (value << 16);
             }
         }
     }

com.unity.render-pipelines.core/Runtime/Utilities/CameraCaptureBridge.cs

@@ -35,7 +35,7 @@ public static bool enabled
         /// <returns>Enumeration of actions</returns>
         public static IEnumerator<Action<RenderTargetIdentifier, CommandBuffer>> GetCaptureActions(Camera camera)
         {
-            if (!actionDict.TryGetValue(camera, out var actions))
+            if (!actionDict.TryGetValue(camera, out var actions) || actions.Count == 0)
                 return null;
 
             return actions.GetEnumerator();

com.unity.render-pipelines.core/Runtime/Volume/VolumeComponent.cs

@@ -219,6 +219,19 @@ public override int GetHashCode()
             }
         }
 
+        /// <summary>
+        /// Returns true if any of the volume properites has been overridden.
+        /// </summary>
+        /// <returns>True if any of the volume properites has been overridden.</returns>
+        public bool AnyPropertiesIsOverridden()
+        {
+            for (int i = 0; i < parameters.Count; ++i)
+            {
+                if (parameters[i].overrideState) return true;
+            }
+            return false;
+        }
+
         /// <summary>
         /// Unity calls this method before the object is destroyed.
         /// </summary>

com.unity.render-pipelines.high-definition/CHANGELOG.md

@@ -19,6 +19,17 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
 - Fixed HDRP Decals performances when they use different materials (~5x improvement in the decal update loop code).
 - Fixed light unit conversion after changing mid gray value.
 - Fixed stencil buffer resolve when MSAA is enabled so that OR operator is used instead of picking the last sample.
+- Fixed rasterized accumulation motion blur when DoF is enabled (case 1378497).
+- Fixed light mode not available after switching a light to area "Disc" or "Tube" (case 1372588).
+- Fixed CoC size computation when dynamic resolution is enabled
+- Fixed shadow cascade transition not working properly with bias.
+- Fixed broken rendering when duplicating a camera while the Rendering Debugger is opened.
+- Fixed screen space shadow debug view not showing when no shadows is available.
+- Fixed debug window reset.
+- Fixed camera bridge action in release build (case 1367866).
+- Fixed contact shadow disappearing when shadowmask is used and no non-static object is available.
+- Fixed atmospheric scattering being incorrectly enabled when scene lighting is disabled.
+- Fixed for changes of color curves not being applied immediately.
 
 ### Changed
 - MaterialReimporter.ReimportAllMaterials and MaterialReimporter.ReimportAllHDShaderGraphs now batch the asset database changes to improve performance.

com.unity.render-pipelines.high-definition/Editor/Lighting/HDLightUI.cs

@@ -223,7 +223,7 @@ static void DrawGeneralContent(SerializedHDLight serialized, Editor owner)
             EditorGUI.showMixedValue = false;
 
             // Draw the mode, for Tube and Disc lights, there is only one choice, so we can disable the enum.
-            using (new EditorGUI.DisabledScope(serialized.areaLightShape == AreaLightShape.Tube || serialized.areaLightShape == AreaLightShape.Disc))
+            using (new EditorGUI.DisabledScope(updatedLightType == HDLightType.Area && (serialized.areaLightShape == AreaLightShape.Tube || serialized.areaLightShape == AreaLightShape.Disc)))
                 serialized.settings.DrawLightmapping();
 
             if (updatedLightType == HDLightType.Area)

com.unity.render-pipelines.high-definition/Runtime/Lighting/AtmosphericScattering/AtmosphericScattering.hlsl

@@ -257,7 +257,7 @@ void EvaluateAtmosphericScattering(PositionInputs posInput, float3 V, out float3
 
 #ifdef DEBUG_DISPLAY
     // Don't sample atmospheric scattering when lighting debug more are enabled so fog is not visible
-    if (_DebugLightingMode >= DEBUGLIGHTINGMODE_DIFFUSE_LIGHTING && _DebugLightingMode <= DEBUGLIGHTINGMODE_EMISSIVE_LIGHTING)
+    if (_DebugLightingMode == DEBUGLIGHTINGMODE_MATCAP_VIEW || (_DebugLightingMode >= DEBUGLIGHTINGMODE_DIFFUSE_LIGHTING && _DebugLightingMode <= DEBUGLIGHTINGMODE_EMISSIVE_LIGHTING))
         return;
 
     if (_DebugShadowMapMode == SHADOWMAPDEBUGMODE_SINGLE_SHADOW || _DebugLightingMode == DEBUGLIGHTINGMODE_LUX_METER || _DebugLightingMode == DEBUGLIGHTINGMODE_LUMINANCE_METER)

com.unity.render-pipelines.high-definition/Runtime/Lighting/LightDefinition.cs

@@ -105,7 +105,10 @@ struct DirectionalLightData
         public float   distanceFromCamera;      // -1 -> no sky interaction
         public float   angularDiameter;         // Units: radians
         public float   flareFalloff;
-        public float   __unused__;
+
+        public float   flareCosInner;
+        public float   flareCosOuter;
+        public float   __unused__;		
 
         public Vector3 flareTint;
         public float   flareSize;               // Units: radians