commit 88bada3c32ba3f1d53073a003085131d60b09213
author John McCutchan <john@johnmccutchan.com> Wed Jul 26 07:47:08 2023 -0700
committer GitHub <noreply@github.com> Wed Jul 26 07:47:08 2023 -0700
tree 7b1e6f93042018adc24147391df932b5dd4c9bf3
parent 048777a93ecc2918cdb6e0353359e6e548cb907b

Revert "Fix rotation around Y axis (#262)" (#300)

This reverts commit 048777a93ecc2918cdb6e0353359e6e548cb907b.

lib/src/vector_math/matrix3.dart

diff --git a/lib/src/vector_math/matrix3.dart b/lib/src/vector_math/matrix3.dart
index cea8dd0..c2b1c83 100644
--- a/lib/src/vector_math/matrix3.dart
+++ b/lib/src/vector_math/matrix3.dart
@@ -556,11 +556,11 @@
     final s = math.sin(radians);
     _m3storage[0] = c;
     _m3storage[1] = 0.0;
-    _m3storage[2] = -s;
+    _m3storage[2] = s;
     _m3storage[3] = 0.0;
     _m3storage[4] = 1.0;
     _m3storage[5] = 0.0;
-    _m3storage[6] = s;
+    _m3storage[6] = -s;
     _m3storage[7] = 0.0;
     _m3storage[8] = c;
   }

lib/src/vector_math_64/matrix3.dart

diff --git a/lib/src/vector_math_64/matrix3.dart b/lib/src/vector_math_64/matrix3.dart
index 89892d3..f497024 100644
--- a/lib/src/vector_math_64/matrix3.dart
+++ b/lib/src/vector_math_64/matrix3.dart
@@ -556,11 +556,11 @@
     final s = math.sin(radians);
     _m3storage[0] = c;
     _m3storage[1] = 0.0;
-    _m3storage[2] = -s;
+    _m3storage[2] = s;
     _m3storage[3] = 0.0;
     _m3storage[4] = 1.0;
     _m3storage[5] = 0.0;
-    _m3storage[6] = s;
+    _m3storage[6] = -s;
     _m3storage[7] = 0.0;
     _m3storage[8] = c;
   }

test/matrix3_test.dart

diff --git a/test/matrix3_test.dart b/test/matrix3_test.dart
index 4e0c3c9..933c4a0 100644
--- a/test/matrix3_test.dart
+++ b/test/matrix3_test.dart
@@ -194,32 +194,11 @@
 
   relativeTest(rotX.transformed(input), input);
   relativeTest(rotY.transformed(input),
-      Vector3(1.0 / math.sqrt(2.0), 0.0, -1.0 / math.sqrt(2.0)));
+      Vector3(1.0 / math.sqrt(2.0), 0.0, 1.0 / math.sqrt(2.0)));
   relativeTest(rotZ.transformed(input),
       Vector3(1.0 / math.sqrt(2.0), 1.0 / math.sqrt(2.0), 0.0));
 }
 
-void testMatrix3RotationX() {
-  final rotX = Matrix3.rotationX(math.pi / 2);
-  final input = Vector3(0.0, 1.0, 0.0);
-
-  relativeTest(rotX.transformed(input), Vector3(0.0, 0.0, 1.0));
-}
-
-void testMatrix3RotationY() {
-  final rotY = Matrix3.rotationY(math.pi / 2);
-  final input = Vector3(0.0, 0.0, 1.0);
-
-  relativeTest(rotY.transformed(input), Vector3(1.0, 0.0, 0.0));
-}
-
-void testMatrix3RotationZ() {
-  final rotZ = Matrix3.rotationZ(math.pi / 2);
-  final input = Vector3(1.0, 0.0, 0.0);
-
-  relativeTest(rotZ.transformed(input), Vector3(0.0, 1.0, 0.0));
-}
-
 void testMatrix3Transform2() {
   final rotZ = Matrix3.rotationZ(math.pi / 4);
   final trans = Matrix3(1.0, 0.0, 3.0, 0.0, 1.0, 2.0, 3.0, 2.0, 1.0);
@@ -355,9 +334,6 @@
     test('transform 2D', testMatrix3Transform2);
     test('rotation 2D', testMatrix3AbsoluteRotate2);
     test('transform', testMatrix3Transform);
-    test('rotation 3D x', testMatrix3RotationX);
-    test('rotation 3D y', testMatrix3RotationY);
-    test('rotation 3D z', testMatrix3RotationZ);
     test('constructor', testMatrix3ConstructorCopy);
     test('inversion', testMatrix3Inversion);
     test('dot product', testMatrix3Dot);