Make rotation math more comprehensible

aardvark-platform · Sep 12, 2024 · 32dedc3 · 32dedc3
1 parent fb0e539
commit 32dedc3
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/src/Aardvark.OpenCV/ImageProcessing.cs b/src/Aardvark.OpenCV/ImageProcessing.cs
@@ -182,7 +182,7 @@ public PixImage<T> Rotate<T>(PixImage<T> image, double angleInRadians, bool resi
                 // Compute bounds of rotated image
                 // See: https://stackoverflow.com/questions/3231176/how-to-get-size-of-a-rotated-rectangle
                 var cos = rotMat.At<double>(0, 0);
-                var sin = rotMat.At<double>(1, 0);
+                var sin = rotMat.At<double>(0, 1);
                 var cosAbs = cos.Abs();
                 var sinAbs = sin.Abs();
                 dstSize.X = (long)(image.Width * cosAbs + image.Height * sinAbs + 0.5);
@@ -193,8 +193,8 @@ public PixImage<T> Rotate<T>(PixImage<T> image, double angleInRadians, bool resi
                 // Shift by -dstCenter -> rotate CW -> shift by srcCenter.
                 // See: https://math.stackexchange.com/questions/2093314/rotation-matrix-of-rotation-around-a-point-other-than-the-origin
                 var dstCenter = dstSize.XY.ToV2d() * 0.5;
-                rotMat.At<double>(0, 2) = -dstCenter.X * cos + dstCenter.Y * sin + srcCenter.X;
-                rotMat.At<double>(1, 2) = -dstCenter.X * sin - dstCenter.Y * cos + srcCenter.Y;
+                rotMat.At<double>(0, 2) = -dstCenter.X * cos - dstCenter.Y * sin + srcCenter.X;
+                rotMat.At<double>(1, 2) =  dstCenter.X * sin - dstCenter.Y * cos + srcCenter.Y;
             }
 
             var dst = dstSize.CreateImageVolume<T>();