fix: fixing a bug with probe colors not being set, also randomizing ProbeProperties color queue

Author: dbirman

Assets/Scripts/Pinpoint/Probes/ProbeManager.cs

@@ -180,6 +180,7 @@ public Color Color
             UIUpdateEvent.Invoke();
             if (ActiveProbeManager == this)
                 ActiveProbeUIUpdateEvent.Invoke();
+            Debug.Log(_color);
         }
     }
 
@@ -216,6 +217,10 @@ private async void Awake()
         UUID = Guid.NewGuid().ToString();
         UpdateName();
 
+        // Set color
+        if (_probeRenderer != null)
+            _color = ProbeProperties.NextColor;
+
         // Record default materials
         _defaultMaterials = new Dictionary<GameObject, Material>();
         _activeRenderers = new();
@@ -243,10 +248,6 @@ private async void Awake()
 
         _axisControl = GameObject.Find("AxisControl").GetComponent<AxisControl>();
 
-        // Set color
-        if (_probeRenderer != null)
-            _color = ProbeProperties.NextColor;
-
         _probeController.FinishedMovingEvent.AddListener(UpdateName);
         _probeController.MovedThisFrameEvent.AddListener(ProbeMoved);
     }
@@ -262,8 +263,6 @@ private void Start()
         if (_probeRenderer) _probeRenderer.material.color = _color;
 
         UIUpdateEvent.Invoke();
-        if (ActiveProbeManager == this)
-            ActiveProbeUIUpdateEvent.Invoke();
     }
 
     /// <summary>

Assets/Scripts/Pinpoint/Probes/ProbeProperties.cs

@@ -27,7 +27,19 @@ public class ProbeProperties
         new Color(0.6196078431372549f, 0.8549019607843137f, 0.8980392156862745f, 1.0f)
     };
 
-    private static Queue<int> _colorIdxQueue = new Queue<int>(Enumerable.Range(0, ProbeColors.Count));
+    private static Queue<int> _colorIdxQueue;
+
+    public static void InitializeColors()
+    {
+        List<int> idxList = Enumerable.Range(0, ProbeColors.Count).ToList();
+        _colorIdxQueue = new Queue<int>();
+        while (idxList.Count() > 0)
+        {
+            int idx = Random.Range(0, idxList.Count() - 1);
+            _colorIdxQueue.Enqueue(idxList[idx]);
+            idxList.RemoveAt(idx);
+        }
+    }
 
     /// <summary>
     /// Get the next Probe Color

Assets/Scripts/Pinpoint/Probes/UI/ProbeUIManager.cs

@@ -49,9 +49,6 @@ private void Awake()
 
         GameObject main = GameObject.Find("main");
 
-        // Set color properly
-        UpdateColors();
-
         // Set probe to be un-selected
         ProbeSelected(false);
 

Assets/Scripts/Pinpoint/TrajectoryPlannerManager.cs

@@ -136,6 +136,7 @@ private void Awake()
 #if UNITY_WEBGL && !UNITY_EDITOR
         WebGLInput.captureAllKeyboardInput = true;
 #endif
+            ProbeProperties.InitializeColors();
 
             SetProbeControl(false);
 

Assets/Scripts/Pinpoint/UI/BregmaLambdaBehavior.cs

@@ -28,7 +28,6 @@ public void ResetBLDistance()
     /// <param name="blDistance"></param>
     public void SetBLRange(float min, float max, float blDistance)
     {
-        Debug.Log("Range set");
         _blDistance = blDistance;
 
         _blSlider.minValue = min;
@@ -38,7 +37,6 @@ public void SetBLRange(float min, float max, float blDistance)
 
     public void SetText(float value)
     {
-        Debug.Log("Text set");
         _sliderText.text = $"{Mathf.RoundToInt(value * 100f) / 100f}";
     }