Skip to content

Move contrib charuco to main objdetect #3394

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

Conversation

AleksandrPanov
Copy link
Contributor

@AleksandrPanov AleksandrPanov commented Dec 19, 2022

merge with opencv/opencv#22986

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 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

@AleksandrPanov AleksandrPanov force-pushed the move_contrib_charuco_to_main_objdetect branch 4 times, most recently from bcd5a7b to 8e5619f Compare December 20, 2022 15:29
@AleksandrPanov AleksandrPanov force-pushed the move_contrib_charuco_to_main_objdetect branch from 8e5619f to 2a3e6b7 Compare December 21, 2022 00:00
@AleksandrPanov AleksandrPanov marked this pull request as ready for review December 21, 2022 00:49
@AleksandrPanov AleksandrPanov force-pushed the move_contrib_charuco_to_main_objdetect branch from 2a3e6b7 to f7677b4 Compare December 21, 2022 00:50
@AleksandrPanov AleksandrPanov force-pushed the move_contrib_charuco_to_main_objdetect branch from f7677b4 to 297724f Compare December 21, 2022 07:17
@AleksandrPanov AleksandrPanov force-pushed the move_contrib_charuco_to_main_objdetect branch from e89ae2e to 4c3af23 Compare December 21, 2022 15:08
@asmorkalov asmorkalov self-requested a review December 21, 2022 15:25
@AleksandrPanov AleksandrPanov force-pushed the move_contrib_charuco_to_main_objdetect branch from 8806b09 to f4868db Compare December 21, 2022 16:46
@AleksandrPanov AleksandrPanov force-pushed the move_contrib_charuco_to_main_objdetect branch from 85f5956 to 522bba3 Compare December 22, 2022 00:24
@asmorkalov asmorkalov force-pushed the move_contrib_charuco_to_main_objdetect branch 6 times, most recently from 23cd606 to 9e03d92 Compare December 27, 2022 20:39
@asmorkalov asmorkalov force-pushed the move_contrib_charuco_to_main_objdetect branch from 9e03d92 to 49bdf7b Compare December 27, 2022 20:43
asmorkalov pushed a commit to opencv/opencv that referenced this pull request Dec 28, 2022
…_main_objdetect

merge with opencv/opencv_contrib#3394

move Charuco API from contrib to main repo:

- add CharucoDetector:
```
CharucoDetector::detectBoard(InputArray image, InputOutputArrayOfArrays markerCorners, InputOutputArray markerIds, 
                             OutputArray charucoCorners, OutputArray charucoIds) const // detect charucoCorners and/or markerCorners
CharucoDetector::detectDiamonds(InputArray image, InputOutputArrayOfArrays _markerCorners,
                                InputOutputArrayOfArrays _markerIds, OutputArrayOfArrays _diamondCorners,
                                OutputArray _diamondIds) const
```

- add `matchImagePoints()` for `CharucoBoard`
- remove contrib aruco dependencies from interactive-calibration tool
- move almost all aruco tests to objdetect

### Pull Request Readiness Checklist

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

- [x] I agree to contribute to the project under Apache 2 License.
- [x] 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
- [x] The PR is proposed to the proper branch
- [x] There is a reference to the original bug report and related work
- [x] There is accuracy test, performance test and test data in opencv_extra repository, if applicable
      Patch to opencv_extra has the same branch name.
- [x] The feature is well documented and sample code can be built with the project CMake
@asmorkalov asmorkalov merged commit e247b68 into opencv:4.x Dec 28, 2022
@alalek alalek mentioned this pull request Jan 8, 2023
a-sajjad72 pushed a commit to a-sajjad72/opencv that referenced this pull request Mar 30, 2023
…uco_to_main_objdetect

merge with opencv/opencv_contrib#3394

move Charuco API from contrib to main repo:

- add CharucoDetector:
```
CharucoDetector::detectBoard(InputArray image, InputOutputArrayOfArrays markerCorners, InputOutputArray markerIds, 
                             OutputArray charucoCorners, OutputArray charucoIds) const // detect charucoCorners and/or markerCorners
CharucoDetector::detectDiamonds(InputArray image, InputOutputArrayOfArrays _markerCorners,
                                InputOutputArrayOfArrays _markerIds, OutputArrayOfArrays _diamondCorners,
                                OutputArray _diamondIds) const
```

- add `matchImagePoints()` for `CharucoBoard`
- remove contrib aruco dependencies from interactive-calibration tool
- move almost all aruco tests to objdetect

### Pull Request Readiness Checklist

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

- [x] I agree to contribute to the project under Apache 2 License.
- [x] 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
- [x] The PR is proposed to the proper branch
- [x] There is a reference to the original bug report and related work
- [x] There is accuracy test, performance test and test data in opencv_extra repository, if applicable
      Patch to opencv_extra has the same branch name.
- [x] The feature is well documented and sample code can be built with the project CMake
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.

2 participants