Skip to content
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

[calibration] Checkerboard detection #1331

Merged
merged 9 commits into from
Apr 12, 2023
Merged

Conversation

mugulmd
Copy link
Contributor

@mugulmd mugulmd commented Jan 13, 2023

Description

This PR introduces a new checkerboard detection software and improves the previous checkerboard detection algorithm.

The detection code was cherry-picked from this PR: #1291.

The corresponding Meshroom node has been introduced in this PR: alicevision/Meshroom#1869.

Features list

  • Robust checkboard detection
  • Support for nested calibration grids
  • Detected boards visualization on top of input images
  • Checkerboard detection software

Tests

Tested on:

  • standard datasets with calibration grid directly facing the camera
  • calibration grids not directly facing the camera and seen under different tilt conditions
  • images taken with an anamorphic lens
  • nested calibration grids.

@mugulmd mugulmd self-assigned this Jan 13, 2023
@mugulmd mugulmd marked this pull request as ready for review January 18, 2023 15:24
@simogasp simogasp added this to the 2.5.0 milestone Jan 20, 2023
@mugulmd mugulmd force-pushed the mug/checkerboardDetection branch from 0c7d553 to 4efca48 Compare March 14, 2023 09:51
@mugulmd
Copy link
Contributor Author

mugulmd commented Mar 14, 2023

Rebased on develop and re-tested :)

@fabiencastan fabiencastan modified the milestones: 3.2.0, 3.1.0 Apr 3, 2023

// write the json file with the tree
ALICEVISION_LOG_INFO("Writing detection output in " << "checkers_" << viewId << ".json");
std::stringstream ss;
Copy link
Member

Choose a reason for hiding this comment

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

boost filesystem path

@fabiencastan fabiencastan merged commit e97ba3a into develop Apr 12, 2023
@fabiencastan fabiencastan deleted the mug/checkerboardDetection branch April 12, 2023 22:13
cbentejac added a commit that referenced this pull request Apr 17, 2023
Following the merge of #1331, the `boost-json` module is needed and thus
needs to be installed along all the other Boost modules.
cbentejac added a commit that referenced this pull request Apr 19, 2023
Following the merge of #1331, the `boost-json` module is needed and thus
needs to be installed along all the other Boost modules.
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.

4 participants