Skip to content

core: expose rectangle intersection to bindings#22526

Merged
opencv-pushbot merged 1 commit intoopencv:4.xfrom
paroj:pyrect
Oct 13, 2022
Merged

core: expose rectangle intersection to bindings#22526
opencv-pushbot merged 1 commit intoopencv:4.xfrom
paroj:pyrect

Conversation

@paroj
Copy link
Copy Markdown
Contributor

@paroj paroj commented Sep 17, 2022

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

@paroj paroj changed the base branch from master to 4.x September 17, 2022 11:03
mshabunin
mshabunin reviewed Sep 18, 2022
View reviewed changes
@paroj paroj requested a review from alalek October 4, 2022 13:10
@fengyuentau
Copy link
Copy Markdown
Member

fengyuentau commented Oct 7, 2022

Could we expose the rectangle union as well? I could see there is a great demand when it comes to calculatoing IoU (intersection over union) in object detection. And these two APIs could help simplify the calculation.

@asmorkalov
Copy link
Copy Markdown
Contributor

asmorkalov commented Oct 12, 2022

@paroj Please fix warnings on CI and I'll merge the patch:

modules/core/include/opencv2/core/types.hpp:2021: trailing whitespace.
+ *

@opencv-pushbot opencv-pushbot merged commit 2763f98 into opencv:4.x Oct 13, 2022
@paroj paroj deleted the pyrect branch October 13, 2022 11:54
@alalek alalek mentioned this pull request Jan 8, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@asmorkalov asmorkalov asmorkalov left review comments

@alalek alalek alalek left review comments

@mshabunin mshabunin mshabunin approved these changes

Assignees

@mshabunin mshabunin

Labels

category: core category: python bindings

Projects

None yet

Milestone

4.7.0

Development

Successfully merging this pull request may close these issues.

6 participants

@paroj @fengyuentau @asmorkalov @mshabunin @alalek @opencv-pushbot