feature: submodule or a class scope for exported classes#21488
Merged
opencv-pushbot merged 1 commit intoopencv:3.4from Mar 2, 2022
Merged
feature: submodule or a class scope for exported classes#21488opencv-pushbot merged 1 commit intoopencv:3.4from
opencv-pushbot merged 1 commit intoopencv:3.4from
Conversation
766d034 to
e8a575e
Compare
c982d92 to
8054f2a
Compare
Member
|
@VadimLevin Thank you! Overall looks good! |
8054f2a to
169993f
Compare
6 tasks
Contributor
Author
After redundant |
169993f to
6d18ae2
Compare
All classes are registered in the scope that corresponds to C++ namespace or exported class. Example: `cv::ml::Boost` is exported as `cv.ml.Boost` `cv::SimpleBlobDetector::Params` is exported as `cv.SimpleBlobDetector.Params` For backward compatibility all classes are registered in the global module with their mangling name containing scope information. Example: `cv::ml::Boost` has `cv.ml_Boost` alias to `cv.ml.Boost` type
6d18ae2 to
ccebbbc
Compare
Merged
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
All classes are registered in the scope that corresponds to C++ namespace or exported class.
Example:
cv::ml::Boostis exported ascv.ml.Boostcv::SimpleBlobDetector::Paramsis exported ascv.SimpleBlobDetector.ParamsFor backward compatibility all classes are registered in the global module with their mangling name containing scope information. Example:
cv::ml::Boosthascv.ml_Boostalias tocv.ml.BoosttypeCloses #14730
Backport for 4.x is required to handle GAPI aliases in the right way:
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.