Use nameof operator for DynamicallyShowInInspector attribute

Author: FilipAlg

AGXUnity/Sensor/AmbientMaterial.cs

@@ -58,7 +58,7 @@ public ConfigurationType AmbientType
     /// <summary>
     /// The visibility of objects as defined by the meterological optical range (MOR) of the medium.
     /// </summary>
-    [DynamicallyShowInInspector( "HasVisibility", true )]
+    [DynamicallyShowInInspector( nameof( HasVisibility ), true )]
     [ClampAboveZeroInInspector()]
     [Tooltip( "The visibility of objects as defined by the meterological optical range (MOR) of the medium." )]
     public float Visibility
@@ -79,7 +79,7 @@ public float Visibility
     /// <summary>
     /// The rate of percipitation in the atmosphere.
     /// </summary>
-    [DynamicallyShowInInspector( "HasRate", true )]
+    [DynamicallyShowInInspector( nameof( HasRate ), true )]
     [ClampAboveZeroInInspector( true )]
     [Tooltip( "The rate of percipitation in the atmosphere." )]
     public float Rate
@@ -100,7 +100,7 @@ public float Rate
     /// <summary>
     /// The signal wavelength used to configure the medium.
     /// </summary>
-    [DynamicallyShowInInspector( "HasWavelength", true )]
+    [DynamicallyShowInInspector( nameof( HasWavelength ), true )]
     [ClampAboveZeroInInspector()]
     [Tooltip( "The signal wavelength used to configure the medium." )]
     public float Wavelength
@@ -121,7 +121,7 @@ public float Wavelength
     /// <summary>
     /// A maritimeness parameter used to specify an interpolation value between fine droplet continental radiation fog (0.0) and large droplet maritime fog (1.0).
     /// </summary>
-    [DynamicallyShowInInspector( "HasMaritimeness", true )]
+    [DynamicallyShowInInspector( nameof( HasMaritimeness ), true )]
     [FloatSliderInInspector( 0, 1 )]
     [Tooltip( "A maritimeness parameter used to specify an interpolation value between fine droplet continental radiation fog (0.0) and large droplet maritime fog (1.0)." )]
     public float Maritimeness
@@ -142,7 +142,7 @@ public float Maritimeness
     /// <summary>
     /// A tropicalness parameter used to specify an interpolation value between common rain of smaller drop size (0.0) and tropical rain of larger drop size (1.0).
     /// </summary>
-    [DynamicallyShowInInspector( "HasTropicalness", true )]
+    [DynamicallyShowInInspector( nameof( HasTropicalness ), true )]
     [FloatSliderInInspector( 0, 1 )]
     [Tooltip( "A tropicalness parameter used to specify an interpolation value between common rain of smaller drop size (0.0) and tropical rain of larger drop size (1.0)." )]
     public float Tropicalness

AGXUnity/Sensor/LidarSensor.cs

@@ -103,6 +103,8 @@ public class LidarSensor : ScriptComponent
     [Tooltip( "The Model, or preset, of this Lidar. " +
               "Changing this will assign Model specific properties to this Lidar." )]
     [DisableInRuntimeInspector]
+
+    [InspectorGroupEnd]
     public LidarModelPreset LidarModelPreset
     {
       get => m_lidarModelPreset;
@@ -127,6 +129,7 @@ public LidarModelPreset LidarModelPreset
     public IModelData ModelData { get; private set; } = new OusterData();
 
     [SerializeField]
+    [InspectorGroupBegin(Name="Local Frame Settings")]
     public GameObject LidarFrame;
 
     private bool HasExplicitFrame() => LidarFrame != null;
@@ -135,14 +138,14 @@ public LidarModelPreset LidarModelPreset
     /// Local sensor rotation relative to the parent GameObject transform.
     /// </summary>
     [Tooltip("Local sensor rotation relative to the parent GameObject transform.")]
-    [DynamicallyShowInInspector("HasExplicitFrame", true, true)]
+    [DynamicallyShowInInspector(nameof(HasExplicitFrame), true, true)]
     public Vector3 LocalRotation = Vector3.zero;
 
     /// <summary>
     /// Local sensor offset relative to the parent GameObject transform.
     /// </summary>
     [Tooltip("Local sensor offset relative to the parent GameObject transform.")]
-    [DynamicallyShowInInspector("HasExplicitFrame", true, true)]
+    [DynamicallyShowInInspector(nameof(HasExplicitFrame), true, true)]
     public Vector3 LocalPosition = Vector3.zero;
 
     /// <summary>