-
-
Notifications
You must be signed in to change notification settings - Fork 56.2k
Add zoom factor to interactive calibration tool #22147
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Conversation
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.
👍
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 makes sense to call putText() after the frame's resize.
Or resize input frames after its capture as the first step.
I don't think |
I agreed, that resizing of input frames is a bad idea (just because we don't get intrinsics for original resolution). But I still believe that |
Add zoom factor to interactive calibration tool * add zoom factor * address review comments
Can be useful when the screen is smaller than the image from camera.
Pull Request Readiness Checklist
See details at https://github.com/opencv/opencv/wiki/How_to_contribute#making-a-good-pull-request
Patch to opencv_extra has the same branch name.