Added '@ref' to Camera Calibration and 3D Reconstruction documentation#18872
Added '@ref' to Camera Calibration and 3D Reconstruction documentation#18872IanMaquignaz wants to merge 9 commits intoopencv:3.4from
Conversation
|
Not sure why the build failed... I only modified the Docs which built with the following warnings: Couple of the warnings stem from half of this enum disapearing: |
Save current work and perform step-by-step small updates. There are too many changes on Diff page. |
| However, if not all of the point pairs ( \f$srcPoints_i\f$, \f$dstPoints_i\f$ ) fit the rigid perspective | ||
| transformation (that is, there are some outliers), this initial estimate will be poor. In this case, | ||
| you can use one of the three robust methods. The methods RANSAC, LMeDS and RHO try many different | ||
| you can use one of the three robust methods. The methods RANSAC, LMEDS and RHO try many different |
There was a problem hiding this comment.
In my opinion, when referring to the method we should keep LMeDS/LMedS.
See Least Median of Squares from a tutorial by Zhengyou Zhang on Parameter Estimation Techniques: A Tutorial with Application to Conic Fitting, published in October 1995.
- Add python explanation for erosion and dilation - Add java explanation for erosion and dilation - Restructure and reword specific sections
…Mat_parameters Minimum change to address issue opencv#18837
|
OK, so... code has been rebuilt locally a few times now and code builds fine with OpenCV master. The build issues seen here stem from the rebase ( In Master: In 3.4: The question therefore is should I be rebasing to merge with 3.4? and if yes, how do I approach this? I've identified one USAC function declaration which appears without reference (see below), but thus far my attempts at a build without this header has only produced further errors downstream.
|
|
Need to create 2 PRs:
|
f46557a to
2069535
Compare
If there's no way to merge my current working alterations in master into 3.4 without painstakingly re-editing calid3d.hpp's 3467 lines then I'm not sure I have the time to complete this task. Alternatively, I've pushed my latest rebase revision which compiles locally for 3.4. Though functional, it reintroduces documentation for |
|
@alalek how do you get the opencv-pushbot to rebuilt everything not just the docs? Docs are fine, just have warnings |
should be removed from this patch (because it goes to 3.4 branch).
I reduced testing scope (PR is under active development, so full scope is not really necessary). |
|
@IanMaquignaz Do you have any progress with the patch? |
|
@asmorkalov I have not had a chance to work on this since my last push. That being said, to accomplish the changes suggested by alalek, I need to create a fresh branch and restart. I may have some time next week. |
|
See new PR #19089 |
|
The patch is replaced with more focused PRs for 3.4 and master . |
Added '@ref' to enumerations to enable doxygen checks in Camera Calibration and 3D Reconstruction documentation.
Notes:
@refto enumerations in@codeblocks.Observations (not addressed)
CALIB_FIX_INTRINSIC Fix cameraMatrix? and distCoeffs? so that only R, T, E, and F matrices are estimated.
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.