Skip to content

[BUG FIX] Object detection sample preprocessing#26326

Merged
asmorkalov merged 10 commits intoopencv:5.xfrom
gursimarsingh:object_detection_fixed
Oct 28, 2024
Merged

[BUG FIX] Object detection sample preprocessing#26326
asmorkalov merged 10 commits intoopencv:5.xfrom
gursimarsingh:object_detection_fixed

Conversation

@gursimarsingh
Copy link
Copy Markdown
Contributor

PR resloves #26315 related to incorrect preprocessing for 'Image2BlobParams' in object detection sample.

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

@gursimarsingh gursimarsingh self-assigned this Oct 17, 2024
@gursimarsingh gursimarsingh changed the base branch from 4.x to 5.x October 17, 2024 13:00
@gursimarsingh gursimarsingh marked this pull request as draft October 17, 2024 13:01
@gursimarsingh gursimarsingh marked this pull request as ready for review October 23, 2024 06:29
@asmorkalov asmorkalov added this to the 5.0-alpha milestone Oct 28, 2024
@asmorkalov asmorkalov merged commit 7b0a082 into opencv:5.x Oct 28, 2024
@opencv-alalek
Copy link
Copy Markdown
Contributor

Please cleanup upstream branches in your fork to avoid pushing tons of commits like

Merge remote-tracking branch 'upstream/5.x' into 5.x

Now we have 1 useful commit and 9 useless.
We really don't want to have 1 : 99 in the future.

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.

3 participants