[HDRP 21.2] Backport various PRs (#6926)

* fixed typo and made style guide fixes (#6749)

* style guide fixes (#6750)

* Added menu trail to change atlas resolution (#6755)

* Clarified how to find hdrp in package manager (#6756)

* made style guide fixes (#6762)

* added new instructions for photoshop colour channels (#6769)

* made naming of AO consistent (#6771)

* Removed reference to 3d with extras project (#6781)

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

* Added default value (#6770)

* Added default value

* Fixed merge issue

* undo wrong fix

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

* Merge Hd/bugfix #6791

* Fix warning on baking set renaming #6808

* Renamed the Exposure field to Exposure Compensation in sky volume overrides (case 1392530). #6801

* Refresh baking set scene list when one baking set is removed. #6811

* clarified how to change mode (#6871)

* fixed screenshot annotation colour (#6872)

* fixed formatting (#6873)

* fixed name of Game view (#6874)

* fixed note formatting (#6875)

* new screenshot and fixed table formatting (#6889)

* minor style guide changes (#6904)

* remove italics, note formatting (#6906)

* fixed typo "on following GPUs" > on the following GPUs (#6909)

* Post Process can now be edited in the default frame settings. #6816

* Fixed errors spamming when in player mode due to ray tracing light cluster debug view (case 1390471). #6820

* Fixed missing unit in ray tracing related tooltips and docs (case 1397491). #6822

* [Fogbugz # 1386436] Fixing FPTL failing when near and far are very tight. #6826

* Fixed an issue where forced sky update would not update ambient probe. #6831

* Fixed a null ref exception in Volume Explorer #6834

* Cleaning up FPTL early out for ps4 and PC (#6882)

* Changelog

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

* [HDRP] Fix static sky lighting update when render textures are updated #6863

* Quality update of custom post process (#6856)

* Disable the clouds in the indoor sample scenes (normal and DXR) #6908

* Fixed using the wrong coordinate to compute the sampling direction for the screen space global illumination. #6903

* Update VolumeParameter.cs

* Replaced view lighting tool with light anchor (#6829)

* Replaced view lighting tool with light anchor

* Update View-Lighting-Tool.md

* Apply formatting changes

* revert: Fix cached directional shadows disappearing when going out of shadow range to never reappear #6802

* [RPW][1386964]Fix undo for DebugUI.Enum fields on the rendering debugger. #6632

* More explicit error message when trying to use Shader Graph nodes HDSceneColor, NormalFromHeight, DDX, DDY or DDXY in ray tracing #6254

* [RPW]Fix recovering the correct index for DebugUI.Enum field that are not autoenum (#6901)

* Proper Indoors scene template volume

* Update 004-CloudsFlaresDecals.png

Co-authored-by: emilybrown1 <88374601+emilybrown1@users.noreply.github.com>
Co-authored-by: FrancescoC-unity <43168857+FrancescoC-unity@users.noreply.github.com>
Co-authored-by: anisunity <42026998+anisunity@users.noreply.github.com>
Co-authored-by: JulienIgnace-Unity <julien@unity3d.com>
Co-authored-by: Kleber Garcia <kleber.garcia@unity3d.com>
Co-authored-by: Antoine Lelievre <antoinel@unity3d.com>
Co-authored-by: noreply@unity3d.com <noreply@unity3d.com>
Co-authored-by: alex-vazquez <76204843+alex-vazquez@users.noreply.github.com>
Co-authored-by: Emmanuel Turquin <emmanuel@turquin.org>
Co-authored-by: Rémi Chapelain <remi.chapelain@unity3d.com>

Author: 11 people

TestProjects/HDRP_RuntimeTests/Assets/ReferenceImages/Linear/OSXPlayer/Metal/None/004-CloudsFlaresDecals.png

@@ -6,8 +6,8 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
 
 ## [13.1.6] - 2022-01-14
 
-Version Updated
-The version number for this package has increased due to a version update of a related graphics package.
+### Fixed
+- Fixed undo in for `DebugUI.EnumFields` on the rendering debugger. (case 1386964)
 
 ## [13.1.5] - 2021-12-17
 

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

@@ -15,4 +15,4 @@
 * [Synchronizing shader code and C#](generating-shader-includes.md)
 * [Look Dev](Look-Dev.md)
   * [Environment Library](Look-Dev-Environment-Library.md)
-* [View Lighting Tool](view-lighting-tool.md)
+* [Light Anchor](view-lighting-tool.md)

com.unity.render-pipelines.core/Documentation~/Images/view-lighting-tool-light-anchor0.png

@@ -1,28 +1,17 @@
-## View Lighting tool
+## Light Anchor
 
-The View Lighting tool allows you to set up lighting in camera space. It uses the main Camera (the one that renders to the Game view) to set the orientation of the Light using spherical coordinates, and it uses a target and distance to set the position of the Light.
+The Light Anchor can help to place light sources around subjects, in relation to a Camera and an anchor point. It's particularly effective for cinematic lighting, which often requires multiple light sources orbiting a subject.
 
-Since the View Lighting tool uses camera-space, it is especially useful when setting up lighting for cinematics.
+## Using the Light Anchor Component
 
-To be able to use the view lighting tool, we need to have at least one camera tagged as "MainCamera" (only one will be used).
+To use the Light Anchor, you must set the Tag of at least one Camera to "MainCamera". By default, the Anchor's position will be the same as the position of the GameObject the Light Anchor Component is attached to.
 
-## Using the View Lighting tool
+Use the **Orbit** and **Elevation** to control the orientation of the light, in degrees, relative to the main Camera's and Anchor's positions. If the Light has a Cookie or an IES Profile, use the **Roll** to change their orientation. Use the **Distance** to control how far from the anchor, in meters, you want to place the Light.
 
-The View Lighting tool uses the Light Anchor component to position and orient the Light. When this component is attached to a Light, you can use a custom Scene view gizmo to position the Light, and use the component's Inspector to orient the Light.
+Using the **Anchor Position Override**, you can provide a custom GameObject as an anchor point for the light. This is useful if you want the light to follow a specific GameObject in the Scene.
 
-To get started with the View Lighting tool:
-
-1. In the Scene view or Hierarchy, select a GameObject with an attached Light component.
-2. In the Inspector click **Add Component > Rendering > Light Anchor**.
-3. To use the Light Anchor gizmo, click the custom tool button then, in the drop-down, click **Light Anchor**.<br/>![](Images/view-lighting-tool-gizmo.png)
-4. You can now use the gizmo to move the Light's target. This gizmo also shows you a visualization of the yaw, pitch, and roll from the target to the Light. To change the yaw, pitch, and roll, as well as the distance from the Light to the target, see [Light Anchor](#light-anchor).
-
-### Light Anchor
-
-The Light Anchor controls the Light's orientation and the distance it is from the target position. **Yaw** and **Pitch** control the orientation and **Distance** controls the distance between the Light and the target. If the Light has a cookie or uses [IES](https://docs.unity3d.com/Packages/com.unity.render-pipelines.high-definition@latest/index.html?subfolder=/manual/IES-Profile.html), **Roll** controls the orientation of the cookie or IES profile.
-
-To quickly reset the angle of a knob to zero. Right-click on it.
+You can set a **Position Offset** for this custom Anchor. This is useful if the Transform position of the custom Anchor isn't centered appropriately for the light to orbit correctly around the custom Anchor.
 
 ![](Images/view-lighting-tool-light-anchor0.png)
 
-The Light Anchor component also includes a list of **Presets** that you can use to quickly set the Light's orientation relative to the Game view.
+The Light Anchor component also includes a list of **Presets** that you can use to set the Light's orientation relative to the main Camera.

com.unity.render-pipelines.core/Documentation~/TableOfContents.md

@@ -134,10 +134,52 @@ public DebugStateAttribute(params Type[] types)
     [Serializable, DebugState(typeof(DebugUI.BoolField), typeof(DebugUI.Foldout), typeof(DebugUI.HistoryBoolField))]
     public sealed class DebugStateBool : DebugState<bool> { }
 
+    /// <summary>
+    /// Enums Debug State.
+    /// </summary>
+    [Serializable, DebugState(typeof(DebugUI.EnumField), typeof(DebugUI.HistoryEnumField))]
+    public sealed class DebugStateEnum : DebugState<int>, ISerializationCallbackReceiver
+    {
+        DebugUI.EnumField m_EnumField;
+
+        /// <summary>
+        /// Set the value of the Debug Item.
+        /// </summary>
+        /// <param name="value">Input value.</param>
+        /// <param name="field">Debug Item field.</param>
+        public override void SetValue(object value, DebugUI.IValueField field)
+        {
+            m_EnumField = field as DebugUI.EnumField;
+            base.SetValue(value, field);
+        }
+
+        void UpdateValue()
+        {
+            if (m_EnumField != null)
+            {
+                base.SetValue(value, m_EnumField);
+                // There might be cases that the value does not map the index, look for the correct index
+                var newCurrentIndex = Array.IndexOf(m_EnumField.enumValues, value);
+                if (m_EnumField.currentIndex != newCurrentIndex)
+                    m_EnumField.currentIndex = newCurrentIndex;
+            }
+        }
+
+        /// <summary>
+        /// On Before Serialize Callback
+        /// </summary>
+        public void OnBeforeSerialize() => UpdateValue();
+
+        /// <summary>
+        /// On After Deserialize Callback
+        /// </summary>
+        public void OnAfterDeserialize() => UpdateValue();
+    }
+
     /// <summary>
     /// Integer Debug State.
     /// </summary>
-    [Serializable, DebugState(typeof(DebugUI.IntField), typeof(DebugUI.EnumField), typeof(DebugUI.HistoryEnumField))]
+    [Serializable, DebugState(typeof(DebugUI.IntField))]
     public sealed class DebugStateInt : DebugState<int> { }
 
     /// <summary>

com.unity.render-pipelines.core/Documentation~/View-Lighting-Tool.md

@@ -304,7 +304,7 @@ public sealed class DebugUIDrawerEnumField : DebugUIDrawer
         public override bool OnGUI(DebugUI.Widget widget, DebugState state)
         {
             var w = Cast<DebugUI.EnumField>(widget);
-            var s = Cast<DebugStateInt>(state);
+            var s = Cast<DebugStateEnum>(state);
 
             if (w.indexes == null)
                 w.InitIndexes();

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

@@ -81,7 +81,7 @@ public virtual void End(DebugUI.Widget widget, DebugState state)
         /// <param name="value">Input value.</param>
         protected void Apply(DebugUI.IValueField widget, DebugState state, object value)
         {
-            Undo.RegisterCompleteObjectUndo(state, "Debug Property Change");
+            Undo.RegisterCompleteObjectUndo(state, $"Debug Property '{state.queryPath}' Change");
             state.SetValue(value, widget);
             widget.SetValue(value);
             EditorUtility.SetDirty(state);

com.unity.render-pipelines.core/Editor/Debugging/DebugUIDrawer.Builtins.cs

@@ -422,7 +422,7 @@ void OnGUI()
                         GUI.Toggle(elementRect, m_Settings.selectedPanel == i, panel.displayName, s_Styles.sectionElement);
                         if (EditorGUI.EndChangeCheck())
                         {
-                            Undo.RegisterCompleteObjectUndo(m_Settings, "Debug Panel Selection");
+                            Undo.RegisterCompleteObjectUndo(m_Settings, $"Debug Panel '{panel.displayName}' Selection");
                             var previousPanel = m_Settings.selectedPanel >= 0 && m_Settings.selectedPanel < panels.Count
                                 ? panels[m_Settings.selectedPanel]
                                 : null;

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

@@ -131,8 +131,8 @@ void InitializeBakingSetList()
                         m_RenameSelectedBakingSet = false;
 
                         // Rename profile asset to match name:
-                        set.profile.name = set.name;
                         AssetDatabase.RenameAsset(AssetDatabase.GetAssetPath(set.profile), set.name);
+                        set.profile.name = set.name;
                     }
                 }
                 else
@@ -163,6 +163,9 @@ void InitializeBakingSetList()
                         AssetDatabase.DeleteAsset(pathToDelete);
                     Undo.RegisterCompleteObjectUndo(sceneData.parentAsset, "Deleted baking set");
                     ReorderableList.defaultBehaviours.DoRemoveButton(list);
+                    // A new set will be selected automatically, so we perform the same operations as if we did the selection explicitly.
+                    OnBakingSetSelected(m_BakingSets);
+
                 }
             };