APV: update some tooltips and add a clamp on dilation validity threshold

com.unity.render-pipelines.core/Editor/Lighting/ProbeVolume/ProbeVolumeBakingProcessSettingsDrawer.cs

@@ -8,9 +8,9 @@ class ProbeVolumeBakingProcessSettingsDrawer : PropertyDrawer
     {
         static class Styles
         {
-            public static readonly GUIContent enableDilation = new GUIContent("Enable Dilation", "Whether to enable dilation after the baking.");
+            public static readonly GUIContent enableDilation = new GUIContent("Enable Dilation", "Whether to enable dilation after the baking. Dilation will dilate valid probes data into invalid probes.");
             public static readonly GUIContent dilationDistance = new GUIContent("Dilation Distance", "The distance used to pick neighbouring probes to dilate into the invalid probe.");
-            public static readonly GUIContent dilationValidityDistance = new GUIContent("Dilation Validity Threshold", "The validity threshold used to identify invalid probes.");
+            public static readonly GUIContent dilationValidity = new GUIContent("Dilation Validity Threshold", "The validity threshold used to identify invalid probes.");
             public static readonly GUIContent dilationIterationCount = new GUIContent("Dilation Iteration Count", "The number of times the dilation process takes place.");
             public static readonly GUIContent dilationSquaredDistanceWeighting = new GUIContent("Squared Distance Weighting", "Whether to weight neighbouring probe contribution using squared distance rather than linear distance.");
             public static readonly GUIContent useVirtualOffset = EditorGUIUtility.TrTextContent("Use Virtual Offset", "Push invalid probes out of geometry. Please note, this feature is currently a proof of concept, it is fairly slow and not optimal in quality.");
@@ -57,8 +57,8 @@ void DrawDilationSettings(SerializedProperty dilationSettings)
             enableDilation.boolValue = EditorGUILayout.Toggle(Styles.enableDilation, enableDilation.boolValue);
             EditorGUI.BeginDisabledGroup(!enableDilation.boolValue);
             maxDilationSampleDistance.floatValue = Mathf.Max(EditorGUILayout.FloatField(Styles.dilationDistance, maxDilationSampleDistance.floatValue), 0);
-            dilationValidityThresholdInverted = EditorGUILayout.Slider(Styles.dilationValidityDistance, dilationValidityThresholdInverted, 0f, 1f);
-            dilationValidityThreshold.floatValue = 1f - dilationValidityThresholdInverted;
+            dilationValidityThresholdInverted = EditorGUILayout.Slider(Styles.dilationValidity, dilationValidityThresholdInverted, 0f, 0.95f);
+            dilationValidityThreshold.floatValue = Mathf.Max(0.05f, 1.0f - dilationValidityThresholdInverted);
             dilationIterations.intValue = EditorGUILayout.IntSlider(Styles.dilationIterationCount, dilationIterations.intValue, 1, 5);
             dilationInvSquaredWeight.boolValue = EditorGUILayout.Toggle(Styles.dilationSquaredDistanceWeighting, dilationInvSquaredWeight.boolValue);
             EditorGUI.indentLevel--;

com.unity.render-pipelines.core/Runtime/Lighting/ProbeVolume/ProbeReferenceVolumeProfile.cs

@@ -84,7 +84,7 @@ internal class ProbeReferenceVolumeProfileEditor : Editor
         static class Styles
         {
             // TODO: Better tooltip are needed here.
-            public static readonly GUIContent simplificationLevels = new GUIContent("Simplification levels", "Determine how much bricks there is in a streamable unit.");
+            public static readonly GUIContent simplificationLevels = new GUIContent("Simplification levels", "Determine how many bricks are in a streamable unit. Each simplification step adds a brick level above the minimum one.");
             public static readonly string simplificationLevelsHighWarning = "High simplification levels have a big memory overhead, they are not recommended except for testing purposes.";
             public static readonly GUIContent minDistanceBetweenProbes = new GUIContent("Min Distance Between Probes", "The minimal distance between two probes in meters.");
             public static readonly GUIContent indexDimensions = new GUIContent("Index Dimensions", "The dimensions of the index buffer.");

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

@@ -13,15 +13,15 @@ namespace UnityEngine.Experimental.Rendering
     public sealed class ProbeVolumesOptions : VolumeComponent
     {
         /// <summary>
-        /// The overridden normal bias to be applied to the world position when sampling the Probe Volumes data structure. Units is meters.
+        /// The overridden normal bias to be applied to the world position when sampling the Probe Volumes data structure. Unit is meters.
         /// </summary>
-        [Tooltip("The overridden normal bias to be applied to the world position when sampling the Probe Volumes data structure. Units is meters.")]
+        [Tooltip("The overridden normal bias to be applied to the world position when sampling the Probe Volumes data structure. Unit is meters.")]
         public ClampedFloatParameter normalBias = new ClampedFloatParameter(0.33f, 0.0f, 2.0f);
 
         /// <summary>
-        /// A bias alongside the view vector to be applied to the world position when sampling the Probe Volumes data structure. Units is meters
+        /// A bias alongside the view vector to be applied to the world position when sampling the Probe Volumes data structure. Unit is meters.
         /// </summary>
-        [AdditionalProperty, Tooltip("A bias alongside the view vector to be applied to the world position when sampling the Probe Volumes data structure. Units is meters.")]
+        [AdditionalProperty, Tooltip("A bias alongside the view vector to be applied to the world position when sampling the Probe Volumes data structure. Unit is meters.")]
         public ClampedFloatParameter viewBias = new ClampedFloatParameter(0.0f, 0.0f, 1.0f);
 
         /// <summary>