Add 180 degree rotation to solver attachment points

to account for the updated glTF coordinate space.

Author: keveleigh

Assets/HoloToolkit/Utilities/Scripts/Solvers/SolverHandler.cs

@@ -46,7 +46,7 @@ public TrackedObjectToReferenceEnum TrackedObjectToReference
                 if (trackedObjectToReference != value)
                 {
                     trackedObjectToReference = value;
-                    TransformTarget = null;
+                    OnControllerLost();
                     AttachToNewTrackedObject();
                 }
             }
@@ -66,7 +66,7 @@ public Vector3 AdditionalOffset
             set
             {
                 additionalOffset = value;
-                UpdateOffsetTransform();
+                TrackTransform(TransformTarget);
             }
         }
 
@@ -76,7 +76,7 @@ public Vector3 AdditionalRotation
             set
             {
                 additionalRotation = value;
-                UpdateOffsetTransform();
+                TrackTransform(TransformTarget);
             }
         }
 
@@ -202,24 +202,7 @@ public virtual void AttachToNewTrackedObject()
 
         private void TrackTransform(Transform newTrackedTransform)
         {
-            if (RequiresOffset())
-            {
-                TransformTarget = MakeOffsetTransform(newTrackedTransform);
-            }
-            else
-            {
-                TransformTarget = newTrackedTransform;
-            }
-        }
-
-        private bool RequiresOffset()
-        {
-            return AdditionalOffset.sqrMagnitude != 0 || AdditionalRotation.sqrMagnitude != 0;
-        }
-
-        private void UpdateOffsetTransform()
-        {
-            TransformTarget = MakeOffsetTransform(TransformTarget);
+            TransformTarget = MakeOffsetTransform(newTrackedTransform);
         }
 
         private Transform MakeOffsetTransform(Transform parentTransform)
@@ -233,6 +216,9 @@ private Transform MakeOffsetTransform(Transform parentTransform)
             transformWithOffset.transform.localPosition = AdditionalOffset;
             transformWithOffset.transform.localRotation = Quaternion.Euler(AdditionalRotation);
             transformWithOffset.name = string.Format("{0} on {1} with offset {2}, {3}", gameObject.name, TrackedObjectToReference.ToString(), AdditionalOffset, AdditionalRotation);
+            // In order to account for the reversed normals due to the glTF coordinate system change, we need to provide rotated attachment points.
+            transformWithOffset.transform.Rotate(0, 180, 0);
+
             return transformWithOffset.transform;
         }