You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
According to the [MVG], various least squares method explanations (like this) and the standard camera calibration code in OpenCV (here), the parameters uncertainty should be calculated with the "minus k" denominator correction: $$\sigma^2 = \frac{1}{n - k} \sum^n_{i=1} \left( x_i - \bar{x} \right)^2$$
where n is a number of errors and k is a number of non-fixed model parameters.
MVG: R. Hartley, A. Zisserman, Multiple View Geometry in Computer Vision, 2004, section 5.1.3, page 134
Steps to reproduce
This is the low priority issue since the calculated value is not used in latter calculation.
However, we may extend the function's interface in the future by adding stddevs for estimated parameters, the same way we did for standard pinhole calibration. When doing this, we should be aware of this bug at least.
Issue submission checklist
I report the issue, it's not a question
I checked the problem with documentation, FAQ, open issues, forum.opencv.org, Stack Overflow, etc and have not found any solution
I updated to the latest OpenCV version and the issue is still there
There is reproducer code and related data files (videos, images, onnx, etc)
The text was updated successfully, but these errors were encountered:
Fixes#23057
Parameter uncertainty fixed + ground truth test data fixed
### Pull Request Readiness Checklist
See details at https://github.com/opencv/opencv/wiki/How_to_contribute#making-a-good-pull-request
- [x] I agree to contribute to the project under Apache 2 License.
- [x] To the best of my knowledge, the proposed patch is not based on a code under GPL or another license that is incompatible with OpenCV
- [x] The PR is proposed to the proper branch
- [x] There is a reference to the original bug report and related work
- [x] There is accuracy test, performance test and test data in opencv_extra repository, if applicable
Patch to opencv_extra has the same branch name.
- [x] The feature is well documented and sample code can be built with the project CMake
System Information
OpenCV version: 3.4+
OS / Platform / Compiler: all
Detailed description
Related to #19803
Similar bug in omnidir calibration: #3406@contrib
According to the [MVG], various least squares method explanations (like this) and the standard camera calibration code in OpenCV (here), the parameters uncertainty should be calculated with the "minus k" denominator correction:
$$\sigma^2 = \frac{1}{n - k} \sum^n_{i=1} \left( x_i - \bar{x} \right)^2$$
where n is a number of errors and k is a number of non-fixed model parameters.
The bug appears here.
MVG: R. Hartley, A. Zisserman, Multiple View Geometry in Computer Vision, 2004, section 5.1.3, page 134
Steps to reproduce
This is the low priority issue since the calculated value is not used in latter calculation.
However, we may extend the function's interface in the future by adding stddevs for estimated parameters, the same way we did for standard pinhole calibration. When doing this, we should be aware of this bug at least.
Issue submission checklist
The text was updated successfully, but these errors were encountered: