Aspect ratio not checked in calibration.cpp (Bug #1632)

Added by Pranav Kant Gaur about 13 years ago. Updated almost 13 years ago.

Status:Done Start date:2012-02-29
Priority:Normal Due date:
Assignee:Ilya Lysenkov % Done:

0%
Category:calibration, 3d
Target version:2.4.0
Affected version: Operating System:
Difficulty: HW Platform:
Pull request:

Description

In calibration.cpp:

http://code.opencv.org/projects/opencv/repository/entry/tags/2.3.1/opencv/modules/calib3d/src/calibration.cpp#L1570
line:1570(inside if condition statement):

aspect ratio is not checked to have a valid value(i.e.,between 0.001 to 100).

Associated revisions

Revision 8b63e33b
Added by Andrey Pavlenko over 11 years ago

Merge pull request #1632 from alalek:cmake_find_package_build_dir

History

#1
Updated by Pranav Kant Gaur about 13 years ago

Sorry!! lower limit is 0.01 & not 0.001.

Pranav Kant Gaur wrote:

In calibration.cpp:

http://code.opencv.org/projects/opencv/repository/entry/tags/2.3.1/opencv/modules/calib3d/src/calibration.cpp#L1570
line:1570(inside if condition statement):

aspect ratio is not checked to have a valid value(i.e.,between 0.001 to 100).

#2
Updated by Alexander Shishkov almost 13 years ago

  • Priority changed from High to Normal
  • Target version deleted ()
  • Category set to calibration, 3d

#3
Updated by Ilya Lysenkov almost 13 years ago

  • Assignee set to Ilya Lysenkov

#4
Updated by Ilya Lysenkov almost 13 years ago

Thank you for the report. It is fixed in r7641.

  • Status changed from Open to Done

#5
Updated by Alexander Shishkov almost 13 years ago

  • Target version set to 2.4.0

Also available in: Atom PDF