Merge 8.x.x/hd/staging (#38)

* [8.x.x Backport]Fixed the MaxLightCount being display when the light volume debug menu is on ColorAndEdge. (#6210)

* Fixed the MaxLightCount being display when the light volume debug menu is on ColorAndEdge.

* Update CHANGELOG.md

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

* [8.x.x Backport] Update documentation to mention more explicitely the lack of support of vertex animation (#6204)

* Update documentation to mention more explicitely the lack of support  of vertex animation

* Update Ray-Tracing-Getting-Started.md

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

* [8.x.x Backport] Make subsection in decal debug menu to specify "affect transparents" (#6202)

* Make subsection in decal debug menu

* changelog

* Support cookie on light backing for Disc Area Light #6138

* Hdrp /lookdev/fix crash on adding volume as viewed game object #6129

* fixed changelog

* Add smoothness remark (#6226)

* Removed legacy VR code from HDRP #5923

* Fixed duplicated entry for com.unity.modules.xr in the runtime asmdef file #5895

* Improve light clipping and culling #2 #4711

* Hdrp/fix/decal material ui errors #6225

* Fix z-fighting in sceneview when scene lighting is off (#6213)

* Do not override the depth state of transparent materials in RenderDebugView

* Changelog

* Changelog - add case number

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

* Update CHANGELOG.md

* Mention default post processing in default volumes. (#6233)

* Fix cubemap inspector so that thumbnail for cubemap work #6231

* - Fix an exception in ray tracing that happens if two LOD levels are using the same mesh renderer. (#6197)

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

* [Backport 8.x.x] Fix some typos in the debug menu (#6249)

* Backport renaming

* changelog

* Added debug exposure to light hierarchy debug. (#6216)

# Conflicts:
#	com.unity.render-pipelines.high-definition/CHANGELOG.md

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

* - Rejecting history for ray traced reflections based on a threshold evaluated on the neighborhood of the sampled history. (#6254)

* Clearer naming of tile and cluster categories for environment lights #6227

* Fix texture curve destroy (#6277)

# Conflicts:
#	com.unity.render-pipelines.core/CHANGELOG.md

* Fixed an issue with refraction model and ray traced recursive rendering (case 1198578). (#6270)

* fix for Fogbugz 1214609 - HDRP Wizard addively increases the Light Intesity instead of setting it (#6266)

* Fixed an issue where a dynamic sky changing any frame may not update the ambient probe. #6235

* Add XR setting to control camera jitter for temporal effects #6259

* fix unreachable code in TextureXR.useTexArray (#6260)

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

* Deprecate Utilities namespace #6219

* Fix issue on thumbnail generation so that no error on new project and thumbnail are generated at first import #6265

* Update CHANGELOG.md

* [8.x.x Backport] Partial fix LookDev opened when CoreRP package reimported (#6289)

* Fix null texture usage happening on CoreRP upgrade

* Partially fix garbage collected stylesheet issue.

(Need more work on UIElement side, see case 1228706 )

* Update CHANGELOG.md

* Clean log

* [Backport 8.x.x] Hdrp/fix default volume switch (#6336)

* Fixed an issue where default volume would not update when switching profile.

* Fixed an issue when setting the default volume for the first time where the hdrp asset would not be dirtied

* Changelog

* [Backport 8.x.x] Fixed an issue where AO override would not override specular occlusion (#6339)

* Manual cherry pick of 9162acb50898c948e35475bb1c0d58dbea4217b7

* Changelog

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

* [Backport 8.x.x] Fixed an issue where Volume inspector might not refresh correctly in some cases. (#6335)

* Fixed an issue where Volume inspector might not refresh correctly in some cases.

* changelog

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

* Hdrp/fix material baked emission #5392

* Fixed volume components lifespan in the clipboard #5869

* Hdrp/fix/particle light shadows #6111

* Bugfix: uncached reflection probe cameras were reseting the debug mode #6245

* Fix case in PBR sky where a camera below ground might invalidate result of a camera above #6272

* Hdrp/probe scale 1219246 (#6288)

* Fix when scale down < 0 all direction of box Influence Volume

* Update Change log

* Update CHANGELOG.md

* Split set rendering feature set and API validity check #6301

* Fix Render Texture with XR #6306

* Fix sRGB mismatch with XR SDK #6311

* Fix XR single-pass with Mock HMD plugin #6313

* Fix XR culling with multiple cameras #6314

* Add custom pass opaque rendering error message #6330

* Update Light-Component.md (#6332)

* Update CHANGELOG.md

* Update TextureCurve.cs

* fix merge issue

* Update CHANGELOG.md

* - Fixed an issue related to the envlightdatasrt not being bound in recursive rendering. (#6410)

* First pass of light loop optimizations (still single threaded) (#6367) (#6424)

* Saving 10% off the PrepareLightForGPU

* 18% decrease in cost with this

* Around 12% cost shaving off GetLightData

* Around 6.5% win here

* Faster View matrix flip

* Missing *=-1

* About 7% win in preprocess light data

* Another small batch

* Small cleanup for first optimization pass

* Tiny bit more cleanup

* Address review points

* Add comment

* changelog
# Conflicts:
#	com.unity.render-pipelines.high-definition/CHANGELOG.md

* Catch and handle NRefactory parser exceptions (#6392)

Co-authored-by: vlad-andreev <vlad-andreev@unity3d.com>

* Avoid negative input to sqrt() (#6397)

- The calculation of `k` is not numerically robust when done in FP16 and might generate a negative value.
- This can be observed as rendering artifacts on Mali drivers newer than Bifrost/Valhall R21.
- This commit fixes this by by clamping `k` to [0, inf] before the `sqrt()`.
- On Mali-G77 the clamp compiles down to a free output modifier on the previous instruction, so there's no performance impact.

* Refreshed the GUIDs for additional post processing data to avoid conflicts with PPv2 (#6394)

* Added known issues page (#6447)

* Platform/playstation #6380

* Shadow cascade tooltip fix (when using the metric mode) #6448

* Focus on Decal uses the extends of the projectors #6449

* [8.x.x Backport] Fixed how the area light influence volume is computed to match rasterization. (#6455)

* - Fixed how the area light influence volume is computed to match rasterization.

* Fix an issue with axis order

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

* Fixed usage of light size data that are not available at runtime. (#6490)

* Changed the diffusion profile warning on the material to an info and changed the message to be more precise. (#6481)

* Fix shadow tesselation culling (#6477)

* Fix custom pass culling params

* Fixed shadow tessellation culling

* revert custom pass code

* Add option to disable XR rendering on the camera settings #6372

* Move scene view camera settings to camera settings window #6390

* Corrected typo and changed links to .md (#6414)

* Corrected typo and changed links to .md

* Update Master-Node-Fabric.md

* Hdrp /fix reset of HDAdditionalLightData #6453

* Fix null error when leaving debug menu #6467

* Hdrp /fix drag area width at left of light intensity #6471

* Fix issue with baked reflection probes constantly marked as dirty with auto-bake on #6473

* Fix custom pass depth copy #6484

* Hdrp /lookdev ui fixes #6491

* Expose a debug constant in ShaderLab to make HDRP/Unlit SRP Batcher compatible again. #6496

* fix unused leftover (#6497)

* Fix custom pass test reference images (#6512)

* [8.x.x Backport] Fixed a weird behavior in the scalable settings drawing when the space becomes tiny (1212045). (#6506)

* - Fixed a weird behavior in the scalable settings drawing when the space becomes tiny (1212045).

* Small fix to avoid text overlapping

Co-authored-by: Remi Chapelain <remi.chapelain@unity3d.com>

* Fixed an usage of a a compute buffer not bound (1229964) (#27)

Co-authored-by: Anis <anis@unity3d.com>

* Removed wrongly serialized fields in StaticLightingSky (#6441)

* Fix issues in the post process system with RenderTexture being invalid in some cases. Causing rendering problems. #6480

* Fixed an issue where changing the default volume profile from another inspector would not update the default volume editor. #6493

* Hdrp/docs/glossary f number (#6523)

* Update Glossary.md

* Update Glossary.md

* path validation when creating new volume profile (#36)

* Clamp probes compression factor to 0 #19

Co-authored-by: anisunity <42026998+anisunity@users.noreply.github.com>
Co-authored-by: FrancescoC-unity <43168857+FrancescoC-unity@users.noreply.github.com>
Co-authored-by: skhiat <55133890+skhiat@users.noreply.github.com>
Co-authored-by: Remi Slysz <40034005+RSlysz@users.noreply.github.com>
Co-authored-by: Fabien Houlmann <44069206+fabien-unity@users.noreply.github.com>
Co-authored-by: Evgenii Golubev <EvgeniiG@users.noreply.github.com>
Co-authored-by: Antoine Lelievre <antoinel@unity3d.com>
Co-authored-by: Pavlos Mavridis <pavlos.mavridis@unity3d.com>
Co-authored-by: JulienIgnace-Unity <julien@unity3d.com>
Co-authored-by: fredericv-unity3d <55485372+fredericv-unity3d@users.noreply.github.com>
Co-authored-by: Thomas <Chman@users.noreply.github.com>
Co-authored-by: JordanL8 <lewis.jordan@hotmail.co.uk>
Co-authored-by: vlad-andreev <vlad-andreev@users.noreply.github.com>
Co-authored-by: vlad-andreev <vlad-andreev@unity3d.com>
Co-authored-by: ChristianF-ARM <45175346+ChristianF-ARM@users.noreply.github.com>
Co-authored-by: Tim Cannell <timcannell@unity3d.com>
Co-authored-by: Jussi Knuuttila <jussi.knuuttila@unity3d.com>
Co-authored-by: Remi Chapelain <remi.chapelain@unity3d.com>
Co-authored-by: Anis <anis@unity3d.com>

Author: 19 people

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

@@ -16,24 +16,24 @@ namespace UnityEditor.Rendering
     /// in the inspector:
     /// <code>
     /// using UnityEngine.Rendering;
-    /// 
+    ///
     /// [CustomEditor(typeof(VolumeProfile))]
     /// public class CustomVolumeProfileEditor : Editor
     /// {
     ///     VolumeComponentListEditor m_ComponentList;
-    /// 
+    ///
     ///     void OnEnable()
     ///     {
     ///         m_ComponentList = new VolumeComponentListEditor(this);
     ///         m_ComponentList.Init(target as VolumeProfile, serializedObject);
     ///     }
-    /// 
+    ///
     ///     void OnDisable()
     ///     {
     ///         if (m_ComponentList != null)
     ///             m_ComponentList.Clear();
     ///     }
-    /// 
+    ///
     ///     public override void OnInspectorGUI()
     ///     {
     ///         serializedObject.Update();
@@ -58,6 +58,8 @@ public sealed class VolumeComponentListEditor
         Dictionary<Type, Type> m_EditorTypes; // Component type => Editor type
         List<VolumeComponentEditor> m_Editors;
 
+        int m_CurrentHashCode;
+
         /// <summary>
         /// Creates a new instance of <see cref="VolumeComponentListEditor"/> to use in an
         /// existing editor.
@@ -195,9 +197,12 @@ public void OnGUI()
             if (asset == null)
                 return;
 
-            if (asset.isDirty)
+            // Even if the asset is not dirty, the list of component may have been changed by another inspector.
+            // In this case, only the hash will tell us that we need to refresh.
+            if (asset.isDirty || asset.GetHashCode() != m_CurrentHashCode)
             {
                 RefreshEditors();
+                m_CurrentHashCode = asset.GetHashCode();
                 asset.isDirty = false;
             }
 

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

@@ -56,12 +56,22 @@ public static VolumeProfile CreateVolumeProfile(Scene scene, string targetName)
             {
                 var scenePath = Path.GetDirectoryName(scene.path);
                 var extPath = scene.name;
-                var profilePath = scenePath + "/" + extPath;
+                var profilePath = scenePath + Path.DirectorySeparatorChar + extPath;
 
                 if (!AssetDatabase.IsValidFolder(profilePath))
-                    AssetDatabase.CreateFolder(scenePath, extPath);
+                {
+                    var directories = profilePath.Split(Path.DirectorySeparatorChar);
+                    string rootPath = "";
+                    foreach (var directory in directories)
+                    {
+                        var newPath = rootPath + directory;
+                        if (!AssetDatabase.IsValidFolder(newPath))
+                            AssetDatabase.CreateFolder(rootPath.TrimEnd(Path.DirectorySeparatorChar), directory);
+                        rootPath = newPath + Path.DirectorySeparatorChar;
+                    }
+                }  
 
-                path = profilePath + "/";
+                path = profilePath + Path.DirectorySeparatorChar;
             }
 
             path += targetName + " Profile.asset";

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

@@ -279,5 +279,23 @@ public bool TryGetAllSubclassOf<T>(Type type, List<T> result)
 
             return count != result.Count;
         }
+
+
+        /// <summary>
+        /// A custom hashing function that Unity uses to compare the state of parameters.
+        /// </summary>
+        /// <returns>A computed hash code for the current instance.</returns>
+        public override int GetHashCode()
+        {
+            unchecked
+            {
+                int hash = 17;
+
+                for (int i = 0; i < components.Count; i++)
+                    hash = hash * 23 + components[i].GetHashCode();
+
+                return hash;
+            }
+        }
     }
 }

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

@@ -40,7 +40,7 @@ A face refers to one side of a piece of geometry. The front face is the side of
 
 #### f-number:
 
-The ratio of the focal length to the diameter of the camera lens.
+The ratio of the focal length to the diameter of the camera lens. HDRP technically uses [t-number](https://en.wikipedia.org/wiki/F-number#T-stop), but since Cameras in Unity are optically perfect, f-number and t-number are identical.
 
 <a name="NyquistRate"></a>
 

com.unity.render-pipelines.high-definition/Documentation~/Glossary.md

@@ -45,6 +45,7 @@ public class Styles
             ReorderableList m_BeforeTransparentCustomPostProcesses;
             ReorderableList m_BeforePostProcessCustomPostProcesses;
             ReorderableList m_AfterPostProcessCustomPostProcesses;
+            int m_CurrentVolumeProfileHash;
 
             public void OnGUI(string searchContext)
             {
@@ -207,6 +208,13 @@ void Draw_VolumeInspector()
                 }
                 EditorGUILayout.EndHorizontal();
 
+                // The state of the profile can change without the asset reference changing so in this case we need to reset the editor.
+                if (m_CurrentVolumeProfileHash != asset.GetHashCode() && m_CachedDefaultVolumeProfileEditor != null)
+                {
+                    m_CurrentVolumeProfileHash = asset.GetHashCode();
+                    m_CachedDefaultVolumeProfileEditor = null;
+                }
+
                 Editor.CreateCachedEditor(asset, Type.GetType("UnityEditor.Rendering.VolumeProfileEditor"), ref m_CachedDefaultVolumeProfileEditor);
                 EditorGUIUtility.labelWidth -= 18;
                 bool oldEnabled = GUI.enabled;
@@ -230,13 +238,13 @@ void Draw_VolumeInspector()
                     hdrpAsset.defaultLookDevProfile = newLookDevAsset;
                     EditorUtility.SetDirty(hdrpAsset);
                 }
-                
+
                 if (GUILayout.Button(EditorGUIUtility.TrTextContent("New", "Create a new Volume Profile for default in your default resource folder (defined in Wizard)"), GUILayout.Width(38), GUILayout.Height(18)))
                 {
                     DefaultVolumeProfileCreator.CreateAndAssign(DefaultVolumeProfileCreator.Kind.LookDev);
                 }
                 EditorGUILayout.EndHorizontal();
-                
+
                 Editor.CreateCachedEditor(lookDevAsset, Type.GetType("UnityEditor.Rendering.VolumeProfileEditor"), ref m_CachedLookDevVolumeProfileEditor);
                 EditorGUIUtility.labelWidth -= 18;
                 oldEnabled = GUI.enabled;
@@ -311,7 +319,7 @@ static string GetDefaultName(Kind kind)
             }
             return defaultName;
         }
-        
+
         public static void CreateAndAssign(Kind kind)
         {
             var assetCreator = ScriptableObject.CreateInstance<DefaultVolumeProfileCreator>();

com.unity.render-pipelines.high-definition/Editor/RenderPipeline/Settings/DefaultSettingsPanel.cs

@@ -136,29 +136,55 @@ ScalableSettingSchema schema
         static void MultiField<T>(Rect position, GUIContent[] subLabels, T[] values)
             where T: struct
         {
+            // The number of slots we need to fit into this rectangle
             var length = values.Length;
-            var num = (position.width - (float) (length - 1) * 3f) / (float) length;
-            var position1 = new Rect(position)
-            {
-                width = num
-            };
-            var labelWidth = EditorGUIUtility.labelWidth;
+
+            // Let's compute the space allocated for every field including the label
+            var num = position.width / (float) length;
+
+            // Reset the indentation
             var indentLevel = EditorGUI.indentLevel;
             EditorGUI.indentLevel = 0;
+
+            // Variable to keep track of the current pixel shift in the rectangle we were assigned for this whole section.
+            float pixelShift = 0;
+
+            // Loop through the levels
             for (var index = 0; index < values.Length; ++index)
             {
-                EditorGUIUtility.labelWidth = CalcPrefixLabelWidth(subLabels[index], (GUIStyle) null);
-                if (typeof(T) == typeof(int))
-                    values[index] = (T)(object)EditorGUI.DelayedIntField(position1, subLabels[index], (int)(object)values[index]);
-                else if (typeof(T) == typeof(bool))
-                    values[index] = (T)(object)EditorGUI.Toggle(position1, subLabels[index], (bool)(object)values[index]);
-                else if (typeof(T) == typeof(float))
-                    values[index] = (T)(object)EditorGUI.FloatField(position1, subLabels[index], (float)(object)values[index]);
-                else
-                    throw new ArgumentOutOfRangeException($"<{typeof(T)}> is not a supported type for multi field");
-                position1.x += num + 4f;
+                // Let's first compute what is the width of the label of this scalable setting level
+				// We make sure that the label doesn't go beyond the space available for this scalable setting level
+				var labelWidth = Mathf.Clamp(CalcPrefixLabelWidth(subLabels[index], (GUIStyle)null), 0, num);
+
+                // Draw the Label at the expected position
+                EditorGUI.LabelField(new Rect(position.x + pixelShift, position.y, labelWidth, position.height), subLabels[index]);
+
+                // We need to remove from the position the label size that we've just drawn and shift by it's length
+                pixelShift += labelWidth;
+
+                // The amount of space left for the field
+                float spaceLeft = num - labelWidth;
+
+                // If at least two pixels are left to draw this field, draw it, otherwise, skip
+                if (spaceLeft > 2)
+                {
+                    // Define the rectangle for the field
+                    var fieldSlot = new Rect(position.x + pixelShift, position.y, num - labelWidth, position.height);
+
+                    // Draw the right field depending on its type.
+                    if (typeof(T) == typeof(int))
+                        values[index] = (T)(object)EditorGUI.DelayedIntField(fieldSlot, (int)(object)values[index]);
+                    else if (typeof(T) == typeof(bool))
+                        values[index] = (T)(object)EditorGUI.Toggle(fieldSlot, (bool)(object)values[index]);
+                    else if (typeof(T) == typeof(float))
+                        values[index] = (T)(object)EditorGUI.FloatField(fieldSlot, (float)(object)values[index]);
+                    else
+                        throw new ArgumentOutOfRangeException($"<{typeof(T)}> is not a supported type for multi field");
+                }
+
+                // Shift by the slot that was left for the field
+                pixelShift += spaceLeft;
             }
-            EditorGUIUtility.labelWidth = labelWidth;
             EditorGUI.indentLevel = indentLevel;
         }
 

com.unity.render-pipelines.high-definition/Editor/RenderPipeline/Settings/SerializedScalableSetting.cs

@@ -3150,7 +3150,8 @@ void BuildGPULightListsCommon(HDCamera hdCamera, CommandBuffer cmd)
                     // Note we clear the whole content and not just the header since it is fast enough, happens only in one frame and is a bit more robust
                     // to changes to the inner workings of the lists.
                     // Also, we clear all the lists and to be resilient to changes in pipeline.
-                    ClearLightList(hdCamera, cmd, resources.tileAndClusterData.bigTileLightList);
+                    if (hdCamera.frameSettings.IsEnabled(FrameSettingsField.BigTilePrepass))
+                        ClearLightList(hdCamera, cmd, resources.tileAndClusterData.bigTileLightList);
                     ClearLightList(hdCamera, cmd, resources.tileAndClusterData.lightList);
                     ClearLightList(hdCamera, cmd, resources.tileAndClusterData.perVoxelOffset);
 

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

@@ -133,6 +133,15 @@ private enum SMAAStage
 
         HDRenderPipeline m_HDInstance;
 
+        void FillEmptyExposureTexture()
+        {
+            var tex = new Texture2D(1, 1, TextureFormat.RGHalf, false, true);
+            tex.SetPixel(0, 0, new Color(1f, ColorUtils.ConvertExposureToEV100(1f), 0f, 0f));
+            tex.Apply();
+            Graphics.Blit(tex, m_EmptyExposureTexture);
+            CoreUtils.Destroy(tex);
+        }
+
         public PostProcessSystem(HDRenderPipelineAsset hdAsset, RenderPipelineResources defaultResources)
         {
             m_Resources = defaultResources;
@@ -206,11 +215,7 @@ public PostProcessSystem(HDRenderPipelineAsset hdAsset, RenderPipelineResources
             // TODO: Write a version that uses structured buffer instead of texture to do atomic as Metal doesn't support atomics on textures.
             m_MotionBlurSupportsScattering = m_MotionBlurSupportsScattering && (SystemInfo.graphicsDeviceType != GraphicsDeviceType.Metal);
 
-            var tex = new Texture2D(1, 1, TextureFormat.RGHalf, false, true);
-            tex.SetPixel(0, 0, new Color(1f, ColorUtils.ConvertExposureToEV100(1f), 0f, 0f));
-            tex.Apply();
-            Graphics.Blit(tex, m_EmptyExposureTexture);
-            CoreUtils.Destroy(tex);
+            FillEmptyExposureTexture();
 
             // Initialize our target pool to ease RT management
             m_Pool = new TargetPool();
@@ -287,6 +292,23 @@ public void Cleanup()
             m_FarBokehTileList          = null;
         }
 
+        // In some cases, the internal buffer of render textures might be invalid.
+        // Usually when using these textures with API such as SetRenderTarget, they are recreated internally.
+        // This is not the case when these textures are used exclusively with Compute Shaders. So to make sure they work in this case, we recreate them here.
+        void CheckRenderTexturesValidity()
+        {
+            if (!m_EmptyExposureTexture.rt.IsCreated())
+                FillEmptyExposureTexture();
+
+            HDUtils.CheckRTCreated(m_InternalLogLut.rt);
+            HDUtils.CheckRTCreated(m_TempTexture1024.rt);
+            HDUtils.CheckRTCreated(m_TempTexture32.rt);
+            if (m_KeepAlpha)
+            {
+                HDUtils.CheckRTCreated(m_AlphaTexture.rt);
+            }
+        }
+
         public void BeginFrame(CommandBuffer cmd, HDCamera camera, HDRenderPipeline hdInstance)
         {
             m_HDInstance = hdInstance;
@@ -336,6 +358,8 @@ public void BeginFrame(CommandBuffer cmd, HDCamera camera, HDRenderPipeline hdIn
             m_DitheringFS           = frameSettings.IsEnabled(FrameSettingsField.Dithering);
             m_AntialiasingFS        = frameSettings.IsEnabled(FrameSettingsField.Antialiasing);
 
+            CheckRenderTexturesValidity();
+
             // Handle fixed exposure & disabled pre-exposure by forcing an exposure multiplier of 1
             if (!m_ExposureControlFS)
             {
@@ -2644,7 +2668,11 @@ public RTHandle Get(in Vector2 scaleFactor, GraphicsFormat format, bool mipmap =
                 var hashCode = ComputeHashCode(scaleFactor.x, scaleFactor.y, (int)format, mipmap);
 
                 if (m_Targets.TryGetValue(hashCode, out var stack) && stack.Count > 0)
-                    return stack.Pop();
+                {
+                    var tex = stack.Pop();
+                    HDUtils.CheckRTCreated(tex.rt);
+                    return tex;
+                }
 
                 var rt = RTHandles.Alloc(
                     scaleFactor, TextureXR.slices, DepthBits.None, colorFormat: format, dimension: TextureXR.dimension,

com.unity.render-pipelines.high-definition/Runtime/PostProcessing/PostProcessSystem.cs

@@ -1,3 +1,4 @@
+using System;
 using System.Collections.Generic;
 using UnityEngine.Serialization;
 
@@ -15,8 +16,9 @@ class StaticLightingSky : MonoBehaviour
         int m_LastComputedHash;
         bool m_NeedUpdateStaticLightingSky;
 
-        // This one contain only property values from overridden properties in the original profile component
-        public SkySettings m_SkySettings;
+        [NonSerialized]
+        public SkySettings m_SkySettings; // This one contain only property values from overridden properties in the original profile component
+        [NonSerialized]
         public SkySettings m_SkySettingsFromProfile;
 
         public SkySettings skySettings