feat: adding an original transform to track when custom transforms are in use

Author: dbirman

Assets/Scripts/Core/CoordinateSystems/CoordinateSpaceManager.cs

@@ -5,6 +5,11 @@
 
 public class CoordinateSpaceManager : MonoBehaviour
 {
+    /// <summary>
+    /// Stores the original transform, when the active transform is a custom transform
+    /// </summary>
+    public static CoordinateTransform OriginalTransform;
+
     public static CoordinateSpace ActiveCoordinateSpace;
     public static CoordinateTransform ActiveCoordinateTransform;
     public static CoordinateSpaceManager Instance;

Assets/Scripts/TrajectoryPlanner/Probes/ProbeManager.cs

@@ -932,7 +932,11 @@ public static ProbeData ProbeManager2ProbeData(ProbeManager probeManager)
         data.Angles = probeManager.ProbeController.Insertion.angles;
 
         data.CoordSpaceName = probeManager.ProbeController.Insertion.CoordinateSpace.Name;
-        data.CoordTransformName = probeManager.ProbeController.Insertion.CoordinateTransform.Name;
+
+        if (probeManager.ProbeController.Insertion.CoordinateTransform.Name.Equals("Custom"))
+            data.CoordTransformName = CoordinateSpaceManager.OriginalTransform.Name;
+        else
+            data.CoordTransformName = probeManager.ProbeController.Insertion.CoordinateTransform.Name;
 
         data.SelectionLayerName = probeManager.SelectionLayerName;
 

Assets/Scripts/TrajectoryPlanner/TrajectoryPlannerManager.cs

@@ -914,19 +914,19 @@ private void LoadSavedProbesFromStringArray(string[] savedProbes)
                     CoordinateTransform probeOrigTransform = coordinateTransformOpts[probeData.CoordTransformName];
 
                     ProbeInsertion probeInsertion;
-                    //if (probeOrigTransform.Name != CoordinateSpaceManager.ActiveCoordinateTransform.Name)
-                    //{
-                    //    Debug.LogError($"[TODO] Need to warn user when transforming a probe into the active coordinate space!!");
-                    //    Vector3 newAPMLDV = CoordinateSpaceManager.ActiveCoordinateTransform.Space2Transform(probeOrigTransform.Transform2Space(probeData.APMLDV));
-
-                    //    probeInsertion = new ProbeInsertion(newAPMLDV, probeData.Angles,
-                    //        CoordinateSpaceManager.ActiveCoordinateSpace, CoordinateSpaceManager.ActiveCoordinateTransform);
-                    //}
-                    //else
-                    //{
+                    if (probeOrigTransform.Name != CoordinateSpaceManager.ActiveCoordinateTransform.Name)
+                    {
+                        Debug.LogError($"[TODO] Need to warn user when transforming a probe into the active coordinate space!!");
+                        Vector3 newAPMLDV = CoordinateSpaceManager.ActiveCoordinateTransform.Space2Transform(probeOrigTransform.Transform2Space(probeData.APMLDV));
+
+                        probeInsertion = new ProbeInsertion(newAPMLDV, probeData.Angles,
+                            CoordinateSpaceManager.ActiveCoordinateSpace, CoordinateSpaceManager.ActiveCoordinateTransform);
+                    }
+                    else
+                    {
                         probeInsertion = new ProbeInsertion(probeData.APMLDV, probeData.Angles,
                             CoordinateSpaceManager.ActiveCoordinateSpace, CoordinateSpaceManager.ActiveCoordinateTransform);
-                    //}
+                    }
 
 
                     ProbeManager newProbeManager = AddNewProbe((ProbeProperties.ProbeType)probeData.Type, probeInsertion,