-
Notifications
You must be signed in to change notification settings - Fork 5.7k
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
New rectification and reconstruction functions for omnidir model #3373
base: 4.x
Are you sure you want to change the base?
Conversation
Thank you for the contribution! Could you add tests for the added functionality? BTW, currently this module have zero tests. Please refer on test from other modules, e.g. from here: https://github.com/opencv/opencv_contrib/tree/4.6.0/modules/mcc/test |
@alalek I can add some tests, but similar to the issue opencv/opencv#22537 it is not very easy to test all of it properly. |
I mean to add tests for the added functionality here (similar to your use case). No need to cover by tests already existed functionality - it is out of scope. Also no need to parameterize inputs, one input is enough for good start. |
@alalek sounds good, I will do that! |
@stefan-spiss friendly reminder about the test. |
@asmorkalov thanks and sorry. I was very busy the last couple of weeks, but I am on it and hope I will have the tests ready soon. |
@asmorkalov and @alalek: I implemented a test for the new function Regarding the two other functions I am unsure how to properly test them.
|
dab967f
to
d4d5794
Compare
…, fixed typos in documentation.
To make the 3D reconstruction workflow with the omnidir camera model similar to the one for the pinhole and fisheye model, new functions were implemented:
In addition, the documentation was updated and the naming style for function parameters was unified (new_size -> newSize).
Addresses parts of opencv/opencv#16601 (comment)
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.
Since there was no example code available for 3D reconstruction with the omnidir model, I did not add anything new yet. But if that is required, I of course can add an example and new data for it. For the same reason, no tests were added yet.