-
Notifications
You must be signed in to change notification settings - Fork 149
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
Comments
if possible, for consistency we should use these parameters: 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. |
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. |
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.
The text was updated successfully, but these errors were encountered: