[bluetooth] Use .invert() instead of .inverse() for three's qua…

…ts. Hopefully fixes #195. Thanks to @anicolao for identifying the issue and fix.
cubing · Aug 20, 2022 · e5bb611 · e5bb611
1 parent fa04e69
commit e5bb611
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/cubing/bluetooth/smart-puzzle/gan.ts b/src/cubing/bluetooth/smart-puzzle/gan.ts
@@ -108,7 +108,7 @@ class PhysicalState {
     const quat = new Quaternion(x, y, z, w);
 
     if (!homeQuatInverse) {
-      homeQuatInverse = quat.clone().inverse();
+      homeQuatInverse = quat.clone().invert();
     }
 
     return quat.clone().multiply(homeQuatInverse.clone());

diff --git a/src/cubing/bluetooth/smart-puzzle/gocube.ts b/src/cubing/bluetooth/smart-puzzle/gocube.ts
@@ -104,7 +104,7 @@ export class GoCube extends BluetoothPuzzle {
   }
 
   public resetOrientation(): void {
-    this.homeQuatInverse = this.lastRawQuat.clone().inverse();
+    this.homeQuatInverse = this.lastRawQuat.clone().invert();
     this.currentQuat = new Quaternion(0, 0, 0, 1);
     this.lastTarget = new Quaternion(0, 0, 0, 1);
   }
@@ -140,7 +140,7 @@ export class GoCube extends BluetoothPuzzle {
       this.lastRawQuat = quat.clone();
 
       if (!this.homeQuatInverse) {
-        this.homeQuatInverse = quat.clone().inverse();
+        this.homeQuatInverse = quat.clone().invert();
       }
 
       const targetQuat = quat.clone().multiply(this.homeQuatInverse.clone());