Skip to content

solved issue 2813 #2819

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

Merged
merged 2 commits into from
Jan 22, 2021
Merged

solved issue 2813 #2819

merged 2 commits into from
Jan 22, 2021

Conversation

LaurentBerger
Copy link
Contributor

@LaurentBerger LaurentBerger commented Jan 14, 2021

resolves #2813

Pull Request Readiness Checklist

See details at https://github.com/opencv/opencv/wiki/How_to_contribute#making-a-good-pull-request

  • I agree to contribute to the project under Apache 2 License.
  • To the best of my knowledge, the proposed patch is not based on a code under GPL or other license that is incompatible with OpenCV
  • The PR is proposed to proper branch
  • There is reference to original bug report and related work
  • There is accuracy test, performance test and test data in opencv_extra repository, if applicable
    Patch to opencv_extra has the same branch name.
  • The feature is well documented and sample code can be built with the project CMake

@LaurentBerger
Copy link
Contributor Author

@alalek there is some problem with buildbot?

@@ -113,7 +113,7 @@ class CV_EXPORTS QuasiDenseStereo
* in case of video processing.
* @sa loadParameters
*/
virtual int loadParameters(cv::String filepath) = 0;
CV_WRAP virtual int loadParameters(cv::String filepath) = 0;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like bindings scripts can't properly wrap abstract classes and pure methods.

Copy link
Member

@alalek alalek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well done!

import cv2 as cv

left_img = cv.imread("G:/Lib/opencv/samples/data/aloeL.jpg", cv.IMREAD_COLOR)
right_img = cv.imread("G:/Lib/opencv/samples/data/aloeR.jpg", cv.IMREAD_COLOR)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please remove absolute paths. Use command-line parameters with default values.
Use cv.samples.findFile()

Copy link
Member

@alalek alalek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@LaurentBerger Well done! Thank you for contribution 👍

@alalek alalek merged commit d733a80 into opencv:master Jan 22, 2021
@LaurentBerger LaurentBerger deleted the pyb_qd branch January 22, 2021 09:49
@ZhuLingfeng1993
Copy link

@LaurentBerger
Have you test the correctness of quasi dense stereo algorithm?
#1941 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

No Python bindings for cv::stereo::QuasiDenseStereo
3 participants