Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Expand exposed parameters for camera calibration #467

Open
mleotta opened this issue May 7, 2021 · 2 comments
Open

Expand exposed parameters for camera calibration #467

mleotta opened this issue May 7, 2021 · 2 comments

Comments

@mleotta
Copy link
Member

mleotta commented May 7, 2021

The goal of this task is to expose more OpenCV camera intrinsics constraint parameters to the KWIVER configs so we can edit them in config files. For example, specifying which intrinsic parameters to estimate and which to keep fixed. Also possibly providing an initial guess of intrinsic parameters.

@mleotta mleotta moved this from To do to In progress in Manual Calibration tools May 7, 2021
@mleotta
Copy link
Member Author

mleotta commented May 7, 2021

if possible, for consistency we should use these parameters:
https://github.com/Kitware/kwiver/blob/master/arrows/ceres/options.cxx#L143

We may want to move that camera options class to a more generic commons location that does not depend on Ceres. The MVG arrow is probably a good location. This would avoid duplication of this code for setting similar parameters across implementations.

@borovik135
Copy link
Collaborator

We can try re-using camera configuration parameters specified in ceres/options.cxx, as most of them can be translated to OpenCV camera calibration parameters, but with a "not value", i.e. flags for cv::calibrateCamera are like CALIB_FIX_PRINCIPAL_POINT, CALIB_FIX_ASPECT_RATIO, CALIB_FIX_FOCAL_LENGTH, etc. Camera motion parameters are not applicable to cv::calibrateCamera, so those will not be used for the OpenCV based camera resection procedure.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

No branches or pull requests

2 participants