Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix bugs in checker_detector.cpp after resolving conflicts from PR #3599 #3877

Merged
merged 4 commits into from
Feb 3, 2025

Conversation

gursimarsingh
Copy link
Contributor

This is the pull request for fixing some bugs in "Color Checker detection"

1, Detecting color-checker using Neural network

When loading the pretrained Macbeth color-checker detector model, and set it successfully to CCheckerDetector,

Ptr<CCheckerDetector> detector = CCheckerDetector::create();
    if (!detector->setNet(net))
    {
         cout << "Loading Model failed: Aborting" << endl;
         return 0;
    }

then

if(!detector->process(image, cv::mcc::MCC24, color_checker_roi, 1, true, params)) {
        std::cout<<"Color-checker not found\n";
}
Ptr<mcc::CChecker> checker = detector->getBestColorChecker();
Ptr<CCheckerDraw> cdraw = CCheckerDraw::create(checker);
cdraw->draw(image);
  • problem 1:
    cdraw function failed to draw the correct result due to the following code block failed to update box in checker:
for (Ptr<CChecker> checker : checkers){
  for (cv::Point2f &corner : checker->getBox())
      corner += static_cast<cv::Point2f>(region.tl() + innerRegion.tl());
   ...
}

the corrected and test pass version is:

for (Ptr<CChecker>& checker : checkers){
    std::vector<cv::Point2f> restore_box;
    for (cv::Point2f& corner : checker->getBox()) {
       corner += static_cast<cv::Point2f>(region.tl() + innerRegion.tl());
       restore_box.emplace_back(corner);
    }
   checker->setBox(restore_box);
}
...

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 another license that is incompatible with OpenCV
  • The PR is proposed to the proper branch
  • There is a reference to the 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

@asmorkalov asmorkalov self-assigned this Jan 29, 2025
@asmorkalov asmorkalov merged commit 127ce19 into opencv:4.x Feb 3, 2025
11 checks passed
@asmorkalov asmorkalov added the category: mcc color calibration module label Feb 3, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
category: mcc color calibration module
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants