Transform sync bandwidth optimisations (#990)

Paul Balaji · web-flow · commit b4dda7f3a8e4 · 2019-06-26T12:11:20.000+01:00
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -5,6 +5,11 @@
 ### Breaking Changes
 
 - The constructor for the `ForwardingDispatcher` now accepts a `UnityEngine.LogType` instead of `Improbable.Worker.CInterop.LogLevel` as its parameter. [#987](https://github.com/spatialos/gdk-for-unity/pull/987)
+- The schema for the Transform Synchronization feature module has been optimised to reduce bandwidth. If you use this module, please make use of the updated helper methods and regenerate your snapshot. [#990](https://github.com/spatialos/gdk-for-unity/pull/990)
+    - The `Location` and `Velocity` schema types have been replaced by the `FixedPointVector3` type.
+        - The location and velocity fields are now [`Q21.10`](https://en.wikipedia.org/wiki/Q_(number_format)) fixed point values.
+    - The `Quaternion` schema type has been replaced by the `CompressedQuaternion` type.
+        - Rotation is now compressed from 4 floats to a single uint32.
 
 ### Added
 
diff --git a/UPGRADE_GUIDE.md b/UPGRADE_GUIDE.md
@@ -0,0 +1,34 @@
+# Upgrade Guide
+
+## From `0.2.3` to `Unreleased`
+
+### General changes
+
+When constructing a `ForwardingDispatcher`, you must now provide a `UnityEngine.LogType` instead of a `Improbable.Worker.CInterop.LogLevel`.
+
+### Transform Synchronization changes
+
+Several utility methods in the `TransformUtils` class have been made `internal` to the Transform Synchronization package. We highly recommend that **if you were using the `TransformInternal` component in your logic, to use `UnityEngine.Transform` instead.**
+
+You must no longer use the `TransformInternal.Snapshot` constructor.
+
+```csharp
+// Sample of INVALID code - the `Location` and `Quaternion` schema types have been removed.
+var transform = new TransformInternal.Snapshot
+{
+    Location = new Location((float) coords.X, (float) coords.Y, (float) coords.Z),
+    Rotation = new Quaternion(1, 0, 0, 0),
+    TicksPerSecond = 1f / Time.fixedDeltaTime
+};
+```
+
+Instead, you must use the `TransformUtils.CreateTransformSnapshot` static method. This is to ensure that the contents of an entity's `TransformInternal` component are compressed using the Transform Synchronization module's new compression scheme.
+
+```csharp
+// This is the correct way of initialising a `TransformInternal` snapshot manually.
+var transform = TransformUtils.CreateTransformSnapshot(Coordinates.Zero, Quaternion.identity, Vector3.forward)
+```
+
+If you use the `AddTransformSynchronizationComponents` method in the `TransformSynchronizationHelper` class, you do not need to update code.
+
+**You must regenerate any snapshots which contain entities that make use of the Transform Synchronization feature module.**
diff --git a/snapshots/default.snapshot b/snapshots/default.snapshot
diff --git a/workers/unity/Assets/Playground/Editor/Improbable.Playground.Editor.asmdef b/workers/unity/Assets/Playground/Editor/Improbable.Playground.Editor.asmdef
@@ -3,12 +3,18 @@
     "references": [
         "Improbable.Playground",
         "Improbable.Gdk.Core",
-        "Improbable.Gdk.Generated"
+        "Improbable.Gdk.Generated",
+        "Improbable.Gdk.TransformSynchronization"
     ],
     "optionalUnityReferences": [],
     "includePlatforms": [
         "Editor"
     ],
     "excludePlatforms": [],
-    "allowUnsafeCode": false
+    "allowUnsafeCode": false,
+    "overrideReferences": false,
+    "precompiledReferences": [],
+    "autoReferenced": true,
+    "defineConstraints": [],
+    "versionDefines": []
 }
diff --git a/workers/unity/Assets/Playground/Editor/SnapshotGenerator/SnapshotGenerator.cs b/workers/unity/Assets/Playground/Editor/SnapshotGenerator/SnapshotGenerator.cs
@@ -4,7 +4,6 @@
 using Improbable.Gdk.PlayerLifecycle;
 using Improbable.Gdk.TransformSynchronization;
 using UnityEngine;
-using Quaternion = Improbable.Gdk.TransformSynchronization.Quaternion;
 
 namespace Playground.Editor.SnapshotGenerator
 {
@@ -84,16 +83,9 @@ private static void AddCubeGrid(Snapshot snapshot, int cubeCount)
                         return;
                     }
 
-                    var positionSnapshot = new Position.Snapshot
-                    {
-                        Coords = new Coordinates(x, 1, z)
-                    };
-                    var transformSnapshot = new TransformInternal.Snapshot
-                    {
-                        Location = new Location(x, 1, z),
-                        Rotation = new Quaternion(1, 0, 0, 0),
-                        TicksPerSecond = 1f / Time.fixedDeltaTime
-                    };
+                    var location = new Vector3(x, 1, z);
+                    var positionSnapshot = new Position.Snapshot(location.ToCoordinates());
+                    var transformSnapshot = TransformUtils.CreateTransformSnapshot(location, Quaternion.identity);
 
                     cubeTemplate.SetComponent(positionSnapshot);
                     cubeTemplate.SetComponent(transformSnapshot);
@@ -106,26 +98,20 @@ private static void CreateSpinner(Snapshot snapshot, Coordinates coords)
         {
             const string entityType = "Spinner";
 
-            var transform = new TransformInternal.Snapshot
-            {
-                Location = new Location((float) coords.X, (float) coords.Y, (float) coords.Z),
-                Rotation = new Quaternion(1, 0, 0, 0),
-                TicksPerSecond = 1f / Time.fixedDeltaTime
-            };
+            var transform = TransformUtils.CreateTransformSnapshot(coords, Quaternion.identity);
 
             var template = new EntityTemplate();
-            template.AddComponent(new Position.Snapshot { Coords = coords }, WorkerUtils.UnityGameLogic);
-            template.AddComponent(new Metadata.Snapshot { EntityType = entityType }, WorkerUtils.UnityGameLogic);
+            template.AddComponent(new Position.Snapshot(coords), WorkerUtils.UnityGameLogic);
+            template.AddComponent(new Metadata.Snapshot(entityType), WorkerUtils.UnityGameLogic);
             template.AddComponent(transform, WorkerUtils.UnityGameLogic);
             template.AddComponent(new Persistence.Snapshot(), WorkerUtils.UnityGameLogic);
             template.AddComponent(new Collisions.Snapshot(), WorkerUtils.UnityGameLogic);
-            template.AddComponent(new SpinnerColor.Snapshot { Color = Color.BLUE }, WorkerUtils.UnityGameLogic);
+            template.AddComponent(new SpinnerColor.Snapshot(Color.BLUE), WorkerUtils.UnityGameLogic);
             template.AddComponent(new SpinnerRotation.Snapshot(), WorkerUtils.UnityGameLogic);
 
             template.SetReadAccess(WorkerUtils.UnityGameLogic, WorkerUtils.UnityClient, WorkerUtils.MobileClient);
             template.SetComponentWriteAccess(EntityAcl.ComponentId, WorkerUtils.UnityGameLogic);
 
-
             snapshot.AddEntity(template);
         }
     }
diff --git a/workers/unity/Assets/Playground/Scripts/DisconnectSystem.cs b/workers/unity/Assets/Playground/Scripts/DisconnectSystem.cs
@@ -23,8 +23,7 @@ protected override void OnUpdate()
         {
             Entities.With(group).ForEach((OnDisconnected data) =>
             {
-                Debug.LogWarningFormat("Disconnected from SpatialOS with reason: \"{0}\"",
-                    data.ReasonForDisconnect);
+                Debug.LogWarning($"Disconnected from SpatialOS with reason: {data.ReasonForDisconnect}");
             });
         }
     }
diff --git a/workers/unity/Assets/Playground/Scripts/MonoBehaviours/CheckForPlayerCollision.cs b/workers/unity/Assets/Playground/Scripts/MonoBehaviours/CheckForPlayerCollision.cs
@@ -10,7 +10,7 @@ public class CheckForPlayerCollision : MonoBehaviour
         [Require] private CollisionsWriter collisionWriter;
 #pragma warning restore 649
 
-        void OnTriggerEnter(Collider other)
+        private void OnTriggerEnter(Collider other)
         {
             collisionWriter?.SendPlayerCollidedEvent(new Empty());
         }
diff --git a/workers/unity/Assets/Playground/Scripts/MonoBehaviours/RotationBehaviour.cs b/workers/unity/Assets/Playground/Scripts/MonoBehaviours/RotationBehaviour.cs
@@ -1,7 +1,7 @@
 using Improbable.Gdk.Subscriptions;
 using Improbable.Gdk.TransformSynchronization;
 using UnityEngine;
-using SpatialQuaternion = Improbable.Gdk.TransformSynchronization.Quaternion;
+using SpatialQuaternion = Improbable.Gdk.TransformSynchronization.CompressedQuaternion;
 using Quaternion = UnityEngine.Quaternion;
 
 public class RotationBehaviour : MonoBehaviour
@@ -14,6 +14,6 @@ public class RotationBehaviour : MonoBehaviour
     private void Update()
     {
         transform.rotation *=
-            Quaternion.Euler((RotatingClockWise ? Vector3.up : Vector3.down) * Time.deltaTime * 20.0f);
+            Quaternion.Euler(20.0f * Time.deltaTime * (RotatingClockWise ? Vector3.up : Vector3.down));
     }
 }
diff --git a/workers/unity/Assets/Playground/Scripts/MonoBehaviours/SpawnCubeCommandReceiver.cs b/workers/unity/Assets/Playground/Scripts/MonoBehaviours/SpawnCubeCommandReceiver.cs
@@ -6,7 +6,6 @@
 using Improbable.Gdk.TransformSynchronization;
 using Improbable.Worker.CInterop;
 using UnityEngine;
-using Quaternion = Improbable.Gdk.TransformSynchronization.Quaternion;
 
 namespace Playground.MonoBehaviours
 {
@@ -49,17 +48,14 @@ private void OnEntityIdsReserved(WorldCommands.ReserveEntityIds.ReceivedResponse
                 return;
             }
 
-            var location = transformReader.Data.Location;
+            var location = gameObject.transform.position;
+            location.y += 2;
+
             var cubeEntityTemplate = CubeTemplate.CreateCubeEntityTemplate();
-            cubeEntityTemplate.SetComponent(new Position.Snapshot
-            {
-                Coords = new Coordinates(location.X, location.Y + 2, location.Z)
-            });
-            cubeEntityTemplate.SetComponent(new TransformInternal.Snapshot
-            {
-                Location = new Location(location.X, location.Y + 2, location.Z),
-                Rotation = new Quaternion(1, 0, 0, 0)
-            });
+
+            cubeEntityTemplate.SetComponent(new Position.Snapshot(location.ToCoordinates()));
+            cubeEntityTemplate.SetComponent(TransformUtils.CreateTransformSnapshot(location, Quaternion.identity));
+
             var expectedEntityId = response.FirstEntityId.Value;
 
             worldCommandRequestSender.SendCreateEntityCommand(
diff --git a/workers/unity/Assets/Playground/Scripts/Player/ProcessLaunchCommandSystem.cs b/workers/unity/Assets/Playground/Scripts/Player/ProcessLaunchCommandSystem.cs
@@ -98,8 +98,7 @@ private void HandleLaunchMeRequests()
                 var info = request.Payload;
 
                 rigidbody.AddForceAtPosition(
-                    new Vector3(info.LaunchDirection.X, info.LaunchDirection.Y, info.LaunchDirection.Z) *
-                    info.LaunchEnergy * 100.0f,
+                    info.LaunchEnergy * 100.0f * new Vector3(info.LaunchDirection.X, info.LaunchDirection.Y, info.LaunchDirection.Z),
                     new Vector3(info.ImpactPoint.X, info.ImpactPoint.Y, info.ImpactPoint.Z)
                 );
 
diff --git a/workers/unity/Packages/com.improbable.gdk.transformsynchronization/.schema/improbable/gdk/transform_synchronization/transform_internal.schema b/workers/unity/Packages/com.improbable.gdk.transformsynchronization/.schema/improbable/gdk/transform_synchronization/transform_internal.schema
@@ -1,31 +1,20 @@
 package improbable.gdk.transform_synchronization;
 
-// These are both types to be removed later when we decide on a compression scheme.
-
-type Location {
-    float x = 1;
-    float y = 2;
-    float z = 3;
-}
-
-type Velocity {
-    float x = 1;
-    float y = 2;
-    float z = 3;
+type FixedPointVector3 {
+    sint32 x = 1;
+    sint32 y = 2;
+    sint32 z = 3;
 }
 
-type Quaternion {
-    float w = 1;
-    float x = 2;
-    float y = 3;
-    float z = 4;
+type CompressedQuaternion {
+    uint32 data = 1;
 }
 
 component TransformInternal {
     id = 11000;
-    Location location = 1;
-    Quaternion rotation = 2;
-    Velocity velocity = 3;
+    FixedPointVector3 location = 1;
+    CompressedQuaternion rotation = 2;
+    FixedPointVector3 velocity = 3;
     uint32 physics_tick = 4;
     float ticks_per_second = 5;
 }
diff --git a/workers/unity/Packages/com.improbable.gdk.transformsynchronization/Improbable.Gdk.TransformSynchronization.asmdef b/workers/unity/Packages/com.improbable.gdk.transformsynchronization/Improbable.Gdk.TransformSynchronization.asmdef
@@ -10,5 +10,10 @@
     "optionalUnityReferences": [],
     "includePlatforms": [],
     "excludePlatforms": [],
-    "allowUnsafeCode": false
-}
+    "allowUnsafeCode": true,
+    "overrideReferences": false,
+    "precompiledReferences": [],
+    "autoReferenced": true,
+    "defineConstraints": [],
+    "versionDefines": []
+}
diff --git a/workers/unity/Packages/com.improbable.gdk.transformsynchronization/MonoBehaviours/TransformSynchronization.cs b/workers/unity/Packages/com.improbable.gdk.transformsynchronization/MonoBehaviours/TransformSynchronization.cs
@@ -138,9 +138,9 @@ private void AddCommonReceiveComponents(EntityCommandBuffer commandBuffer)
 
             var defaultToSet = new TransformToSet
             {
-                Position = transformComponent.Location.ToUnityVector3() + world.GetExistingSystem<WorkerSystem>().Origin,
-                Velocity = transformComponent.Velocity.ToUnityVector3(),
-                Orientation = transformComponent.Rotation.ToUnityQuaternion(),
+                Position = TransformUtils.ToUnityVector3(transformComponent.Location) + world.GetExistingSystem<WorkerSystem>().Origin,
+                Velocity = TransformUtils.ToUnityVector3(transformComponent.Velocity),
+                Orientation = TransformUtils.ToUnityQuaternion(transformComponent.Rotation),
                 ApproximateRemoteTick = 0
             };
 
@@ -162,9 +162,9 @@ private void AddCommonSendComponents(EntityCommandBuffer commandBuffer)
 
             var defaultToSend = new TransformToSend
             {
-                Position = transformComponent.Location.ToUnityVector3() - world.GetExistingSystem<WorkerSystem>().Origin,
-                Velocity = transformComponent.Velocity.ToUnityVector3(),
-                Orientation = transformComponent.Rotation.ToUnityQuaternion()
+                Position = TransformUtils.ToUnityVector3(transformComponent.Location) - world.GetExistingSystem<WorkerSystem>().Origin,
+                Velocity = TransformUtils.ToUnityVector3(transformComponent.Velocity),
+                Orientation = TransformUtils.ToUnityQuaternion(transformComponent.Rotation)
             };
 
             var ticksSinceLastUpdate = new TicksSinceLastTransformUpdate
diff --git a/workers/unity/Packages/com.improbable.gdk.transformsynchronization/Systems/DefaultApplyLatestTransformSystem.cs b/workers/unity/Packages/com.improbable.gdk.transformsynchronization/Systems/DefaultApplyLatestTransformSystem.cs
@@ -51,12 +51,11 @@ private void UpdateRigidbodyData()
 
         private void UpdateTransformData()
         {
-            Entities.With(transformGroup)
-                .ForEach((UnityEngine.Transform transform, ref TransformToSet transformToSet) =>
-                {
-                    transform.localPosition = transformToSet.Position;
-                    transform.localRotation = transformToSet.Orientation;
-                });
+            Entities.With(transformGroup).ForEach((Transform transform, ref TransformToSet transformToSet) =>
+            {
+                transform.localPosition = transformToSet.Position;
+                transform.localRotation = transformToSet.Orientation;
+            });
         }
     }
 }
diff --git a/workers/unity/Packages/com.improbable.gdk.transformsynchronization/Systems/DefaultUpdateLatestTransformSystem.cs b/workers/unity/Packages/com.improbable.gdk.transformsynchronization/Systems/DefaultUpdateLatestTransformSystem.cs
@@ -1,5 +1,4 @@
 ﻿using Improbable.Gdk.Core;
-using Improbable.Gdk.TransformSynchronization;
 using Unity.Entities;
 using UnityEngine;
 
@@ -55,7 +54,7 @@ private void UpdateRigidbodyData()
 
         private void UpdateTransformData()
         {
-            Entities.With(transformGroup).ForEach((ref TransformToSend transformToSend, UnityEngine.Transform transform) =>
+            Entities.With(transformGroup).ForEach((ref TransformToSend transformToSend, Transform transform) =>
             {
                 transformToSend = new TransformToSend
                 {
diff --git a/workers/unity/Packages/com.improbable.gdk.transformsynchronization/Systems/DirectTransformUpdateSystem.cs b/workers/unity/Packages/com.improbable.gdk.transformsynchronization/Systems/DirectTransformUpdateSystem.cs
@@ -46,9 +46,9 @@ protected override void OnUpdate()
 
                     transformToSet = new TransformToSet
                     {
-                        Position = transformInternal.Location.ToUnityVector3() + worker.Origin,
-                        Velocity = transformInternal.Velocity.ToUnityVector3(),
-                        Orientation = transformInternal.Rotation.ToUnityQuaternion()
+                        Position = TransformUtils.ToUnityVector3(transformInternal.Location) + worker.Origin,
+                        Velocity = TransformUtils.ToUnityVector3(transformInternal.Velocity),
+                        Orientation = TransformUtils.ToUnityQuaternion(transformInternal.Rotation)
                     };
                 });
         }
diff --git a/workers/unity/Packages/com.improbable.gdk.transformsynchronization/Systems/GetTransformValueToSetSystem.cs b/workers/unity/Packages/com.improbable.gdk.transformsynchronization/Systems/GetTransformValueToSetSystem.cs
@@ -37,15 +37,14 @@ protected override void OnUpdate()
 
                     var bufferHead = buffer[0];
 
-                    var currentTransform = new TransformToSet
+                    transformToSet = new TransformToSet
                     {
                         Position = bufferHead.Position + worker.Origin,
                         Orientation = bufferHead.Orientation,
                         Velocity = bufferHead.Velocity,
                         ApproximateRemoteTick = bufferHead.PhysicsTick
                     };
 
-                    transformToSet = currentTransform;
                     buffer.RemoveAt(0);
                 });
         }
diff --git a/workers/unity/Packages/com.improbable.gdk.transformsynchronization/Systems/InterpolateTransformSystem.cs b/workers/unity/Packages/com.improbable.gdk.transformsynchronization/Systems/InterpolateTransformSystem.cs
@@ -115,7 +115,9 @@ protected override void OnUpdate()
 
                         for (uint k = 0; k < ticksToFill - 1; ++k)
                         {
-                            transformBuffer.Add(InterpolateValues(transformToInterpolateFrom, transformToInterpolateTo,
+                            transformBuffer.Add(InterpolateValues(
+                                transformToInterpolateFrom,
+                                transformToInterpolateTo,
                                 k + 1));
                         }
 
@@ -157,9 +159,9 @@ private static BufferedTransform ToBufferedTransform(TransformInternal.Component
         {
             return new BufferedTransform
             {
-                Position = transform.Location.ToUnityVector3(),
-                Velocity = transform.Velocity.ToUnityVector3(),
-                Orientation = transform.Rotation.ToUnityQuaternion(),
+                Position = TransformUtils.ToUnityVector3(transform.Location),
+                Velocity = TransformUtils.ToUnityVector3(transform.Velocity),
+                Orientation = TransformUtils.ToUnityQuaternion(transform.Rotation),
                 PhysicsTick = transform.PhysicsTick
             };
         }
@@ -168,9 +170,9 @@ private static BufferedTransform ToBufferedTransformAtTick(TransformInternal.Com
         {
             return new BufferedTransform
             {
-                Position = component.Location.ToUnityVector3(),
-                Velocity = component.Velocity.ToUnityVector3(),
-                Orientation = component.Rotation.ToUnityQuaternion(),
+                Position = TransformUtils.ToUnityVector3(component.Location),
+                Velocity = TransformUtils.ToUnityVector3(component.Velocity),
+                Orientation = TransformUtils.ToUnityQuaternion(component.Rotation),
                 PhysicsTick = tick
             };
         }
diff --git a/workers/unity/Packages/com.improbable.gdk.transformsynchronization/Systems/RateLimitedPositionSendSystem.cs b/workers/unity/Packages/com.improbable.gdk.transformsynchronization/Systems/RateLimitedPositionSendSystem.cs
@@ -1,5 +1,4 @@
 using Improbable.Gdk.Core;
-using Improbable.Gdk.TransformSynchronization;
 using Unity.Entities;
 using UnityEngine;
 
diff --git a/workers/unity/Packages/com.improbable.gdk.transformsynchronization/Systems/RateLimitedTransformSendSystem.cs b/workers/unity/Packages/com.improbable.gdk.transformsynchronization/Systems/RateLimitedTransformSendSystem.cs
diff --git a/workers/unity/Packages/com.improbable.gdk.transformsynchronization/Systems/ResetForAuthorityGainedSystem.cs b/workers/unity/Packages/com.improbable.gdk.transformsynchronization/Systems/ResetForAuthorityGainedSystem.cs
diff --git a/workers/unity/Packages/com.improbable.gdk.transformsynchronization/Systems/SetKinematicFromAuthoritySystem.cs b/workers/unity/Packages/com.improbable.gdk.transformsynchronization/Systems/SetKinematicFromAuthoritySystem.cs
diff --git a/workers/unity/Packages/com.improbable.gdk.transformsynchronization/Systems/TickRateEstimationSystem.cs b/workers/unity/Packages/com.improbable.gdk.transformsynchronization/Systems/TickRateEstimationSystem.cs
diff --git a/workers/unity/Packages/com.improbable.gdk.transformsynchronization/TransformSynchronizationHelper.cs b/workers/unity/Packages/com.improbable.gdk.transformsynchronization/TransformSynchronizationHelper.cs
diff --git a/workers/unity/Packages/com.improbable.gdk.transformsynchronization/TransformUtils.cs b/workers/unity/Packages/com.improbable.gdk.transformsynchronization/TransformUtils.cs

Original file line number	Diff line number	Diff line change
`@@ -23,8 +23,7 @@ protected override void OnUpdate()`
`23`	`23`	`{`
`24`	`24`	`Entities.With(group).ForEach((OnDisconnected data) =>`
`25`	`25`	`{`
`26`		`- Debug.LogWarningFormat("Disconnected from SpatialOS with reason: \"{0}\"",`
`27`		`- data.ReasonForDisconnect);`
	`26`	`+ Debug.LogWarning($"Disconnected from SpatialOS with reason: {data.ReasonForDisconnect}");`
`28`	`27`	`});`
`29`	`28`	`}`
`30`	`29`	`}`
Original file line number	Diff line number	Diff line change
`@@ -10,7 +10,7 @@ public class CheckForPlayerCollision : MonoBehaviour`
`10`	`10`	`[Require] private CollisionsWriter collisionWriter;`
`11`	`11`	`#pragma warning restore 649`
`12`	`12`
`13`		`- void OnTriggerEnter(Collider other)`
	`13`	`+ private void OnTriggerEnter(Collider other)`
`14`	`14`	`{`
`15`	`15`	`collisionWriter?.SendPlayerCollidedEvent(new Empty());`
`16`	`16`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1,7 +1,7 @@`
`1`	`1`	`using Improbable.Gdk.Subscriptions;`
`2`	`2`	`using Improbable.Gdk.TransformSynchronization;`
`3`	`3`	`using UnityEngine;`
`4`		`-using SpatialQuaternion = Improbable.Gdk.TransformSynchronization.Quaternion;`
	`4`	`+using SpatialQuaternion = Improbable.Gdk.TransformSynchronization.CompressedQuaternion;`
`5`	`5`	`using Quaternion = UnityEngine.Quaternion;`
`6`	`6`
`7`	`7`	`public class RotationBehaviour : MonoBehaviour`
`@@ -14,6 +14,6 @@ public class RotationBehaviour : MonoBehaviour`
`14`	`14`	`private void Update()`
`15`	`15`	`{`
`16`	`16`	`transform.rotation *=`
`17`		`- Quaternion.Euler((RotatingClockWise ? Vector3.up : Vector3.down) * Time.deltaTime * 20.0f);`
	`17`	`+ Quaternion.Euler(20.0f * Time.deltaTime * (RotatingClockWise ? Vector3.up : Vector3.down));`
`18`	`18`	`}`
`19`	`19`	`}`