-
Notifications
You must be signed in to change notification settings - Fork 732
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
Add Fisheye calibration tool #440
Add Fisheye calibration tool #440
Conversation
camera_calibration_fisheye/fiseye_cal_reference/mono/calibrate.py
Outdated
Show resolved
Hide resolved
Hello |
…66bdeb58c4b138539a64ddea0b17e10
4894735
to
54c271e
Compare
I mange to find some time to look into this: Modifications done. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just come cleanup. I also noticed you changed the permissions on a bunch of files, why is that necessary?
For some reason estimateNewCameraMatrixForUndistortRectify is not producing the expected result, hence a workaround was implemented
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Last issue. Please also articulate how you tested this since there is no test coverage
True: all these features are being tested by live calibration with a real camera. |
Commit bc4f2e3 introduces fisheye calibration flags as input arguments.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK if tested on hardware I'm OK with that.
You broke backwards compatibility with the flag updates, we need another solution to that. I think just no qualifiers is fine since that's what OpenCV does anyhow and fisheye is specified
Rolling back changes to previous commit on camera calibrator flags to enable backwards compatibility
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@JWhitleyAStuff take a look
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See comments.
* Add Fisheye calibration tool * Add calibration for fisheye cameras Fix ros-perception#146 * Correct typo * Restore camera_calib files permisions * Upgrades to calibrator tool for multi model calibration * Solve fisheye balance selection TODO: For some reason estimateNewCameraMatrixForUndistortRectify is not producing the expected result, hence a workaround was implemented * Add fisheye calibration flags as user arguments * Add undistortion of points for fisheye * cam_calib: rolling back flags Rolling back changes to previous commit on camera calibrator flags to enable backwards compatibility * cam_calib: Style formating
* Add Fisheye calibration tool * Add calibration for fisheye cameras Fix #146 * Correct typo * Restore camera_calib files permisions * Upgrades to calibrator tool for multi model calibration * Solve fisheye balance selection TODO: For some reason estimateNewCameraMatrixForUndistortRectify is not producing the expected result, hence a workaround was implemented * Add fisheye calibration flags as user arguments * Add undistortion of points for fisheye * cam_calib: rolling back flags Rolling back changes to previous commit on camera calibrator flags to enable backwards compatibility * cam_calib: Style formating
* Add Fisheye calibration tool * Add calibration for fisheye cameras Fix #146 * Correct typo * Restore camera_calib files permisions * Upgrades to calibrator tool for multi model calibration * Solve fisheye balance selection TODO: For some reason estimateNewCameraMatrixForUndistortRectify is not producing the expected result, hence a workaround was implemented * Add fisheye calibration flags as user arguments * Add undistortion of points for fisheye * cam_calib: rolling back flags Rolling back changes to previous commit on camera calibrator flags to enable backwards compatibility * cam_calib: Style formating
This PR introduces changes to be able to calibrate fisheye cameras and aim to Fix #146
This PR is related with ros/common_msgs#151 , ros-perception/vision_opencv#306 and ros-perception/image_common#146
Main changes it introduces are: