From: Vadim Pisarevsky <no@email>
Date: Mon, 4 Oct 2010 13:46:37 +0000 (+0000)
Subject: possible fix for ticket 380 (should not hurt camera calibration engine, as the fix... 
X-Git-Tag: accepted/2.0/20130307.220821~4424
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=51822f2072268ececcc58da87640631575b588d7;p=profile%2Fivi%2Fopencv.git

possible fix for ticket 380 (should not hurt camera calibration engine, as the fix only affects non-planar objects)
---

diff --git a/modules/calib3d/src/calibration.cpp b/modules/calib3d/src/calibration.cpp
index e4fa225..73efc75 100644
--- a/modules/calib3d/src/calibration.cpp
+++ b/modules/calib3d/src/calibration.cpp
@@ -1279,10 +1279,10 @@ CV_IMPL void cvFindExtrinsicCameraParams2( const CvMat* objectPoints,
             _RRt = cvMat( 3, 4, CV_64F, LV + 11*12 );
             cvGetCols( &_RRt, &_RR, 0, 3 );
             cvGetCol( &_RRt, &_tt, 3 );
-            if( cvDet(&_RR) < 0 )
-                cvScale( &_RRt, &_RRt, -1 );
             sc = cvNorm(&_RR);
             cvSVD( &_RR, &matW, &matU, &matV, CV_SVD_MODIFY_A + CV_SVD_U_T + CV_SVD_V_T );
+            if( W[0]*W[1]*W[2] < 0 )
+                sc = -sc;
             cvGEMM( &matU, &matV, 1, 0, 0, &matR, CV_GEMM_A_T );
             cvScale( &_tt, &_t, cvNorm(&matR)/sc );
             cvRodrigues2( &matR, &_r );