stereoRectifyUncalibrated doesn't accept the same type of array as findFundamentalMat (Bug #2354)


Added by Julien Rebetez over 4 years ago. Updated over 1 year ago.


Status:Open Start date:2012-09-13
Priority:Normal Due date:
Assignee:- % Done:

0%

Category:calibration, 3d
Target version:Next Hackathon
Affected version: Operating System:
Difficulty: HW Platform:
Pull request:

Description

stereoRectifyUncalibrated's documentation states that for the points1 and points2 parameters, the same format as in findFundamentalMat is accepted.
But providing a 2xN or Nx2 array (which are working fine with findFundamentalMat) will trigger the following error :

error: /home/julien/tm/libs/OpenCV/modules/calib3d/src/calibration.cpp:2649: error: (-215) CV_IS_MAT(_points1) && CV_IS_MAT(_points2) && (_points1->rows == 1 || _points1->cols == 1) && (_points2->rows == 1 || _points2->cols == 1) && CV_ARE_SIZES_EQ(_points1, _points2) in function cvStereoRectifyUncalibrated

By looking at the source code, cvStereoRectifyUncalibrated has the following assert :

CV_Assert( CV_IS_MAT(_points1) && CV_IS_MAT(_points2) &&
(_points1->rows 1 || _points1->cols 1) &&
(_points2->rows 1 || _points2->cols 1) &&
CV_ARE_SIZES_EQ(_points1, _points2) );

And cvFindFundamentalMat has the following (which doesn't enforce rows 1 or cols 1) :

CV_Assert( CV_IS_MAT(points1) && CV_IS_MAT(points2) && CV_ARE_SIZES_EQ(points1, points2) );

I also found this discussion which seems to be about this problem :
http://opencv-users.1802565.n2.nabble.com/StereoRectifyUncalibrated-not-accepting-same-array-as-FindFundamentalMat-td5149185.html


History

Updated by Andrey Pavlenko over 4 years ago

  • Assignee set to Vadim Pisarevsky

Updated by Andrey Pavlenko over 4 years ago

  • Assignee changed from Vadim Pisarevsky to Ilya Lysenkov

Updated by Vadim Pisarevsky over 4 years ago

  • Target version deleted ()
  • Assignee deleted (Ilya Lysenkov)

Updated by Kirill Kornyakov over 4 years ago

  • Target version set to Next Hackathon

Updated by Maksim Shabunin over 1 year ago

Issue has been transferred to GitHub: https://github.com/Itseez/opencv/issues/4426

Also available in: Atom PDF