change flags and bugfix graph in new RANSAC. #18483
change flags and bugfix graph in new RANSAC. #18483opencv-pushbot merged 1 commit intoopencv:masterfrom
Conversation
|
The USAC additions are great, but I think the documentation additions are sorely lacking. Briefly looking at the source code, one gets the impression that one can use these more state-of-the-art methods for homographies, essential matrices, fundamental matrices, and PnP, all with the previously existing function signatures (not the new one with the UsacParams struct). However, the documentation of 4.5.0 says next to nothing about this. The consequence is that this great new addition will fly somewhat under the radar until made prominent in the documentation. Or have I misunderstood the USAC additions? |
The documentation of new features indeed could have been better. This was my Google Summer of Code project, so I had a bit lack of time when I was writing it.
There are other minor things but I think everything written above should be enough for USAC to be attractive. Nevertheless, thank you for your comment and feedback. I will rewrite documentation as it is important to make clear what is new and why should someone use it. |
Related: #17683 #18482
Fixes #18454
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.