-
Notifications
You must be signed in to change notification settings - Fork 232
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
Calib update #1083
Calib update #1083
Conversation
* Optimization and trace levels introduced WIP * tweaked parameters and updated tracelevel
* Make sure the same points are present on all images * Use initUndistortRectifyMap from fisheye calib
@@ -129,7 +127,7 @@ def parse_args(): | |||
parser.add_argument("-ih", "--invertHorizontal", dest="invert_h", default=False, action="store_true", | |||
help="Invert horizontal axis of the camera for the display") | |||
parser.add_argument("-ep", "--maxEpiploarError", default="0.7", type=float, required=False, | |||
help="Sets the maximum epiploar allowed with rectification") | |||
help="Sets the maximum epiploar allowed with rectification. Default: %(default)s") |
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.
what is this default set to.?
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.
It is set to 0.7. Should I set it lower?
update this branch to be on-top of main |
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.
LGTM
Clearing the disableRGB,
adding the epipolarThreshold parameter and
minSyncTimestamp parameter for sync of images from different cameras.
Clearing the OAK-D-1 problem with stereo on one camera (line 791)