Skip to content

New HAL entry for cv::sum and IPP adoption#27346

Merged
asmorkalov merged 1 commit intoopencv:4.xfrom
asmorkalov:as/ipp_hal_sum
May 24, 2025
Merged

New HAL entry for cv::sum and IPP adoption#27346
asmorkalov merged 1 commit intoopencv:4.xfrom
asmorkalov:as/ipp_hal_sum

Conversation

@asmorkalov
Copy link
Copy Markdown
Contributor

@asmorkalov asmorkalov commented May 21, 2025

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

@asmorkalov asmorkalov added this to the 4.12.0 milestone May 21, 2025
@asmorkalov asmorkalov changed the title WIP: New HAL entry for cv::sum and IPP adoption New HAL entry for cv::sum and IPP adoption May 23, 2025
opencv-alalek
opencv-alalek reviewed May 23, 2025
View reviewed changes
hal/ipp/src/sum_ipp.cpp
@@ -0,0 +1,55 @@
#include "ipp_hal_core.hpp"
Copy link
Copy Markdown
Contributor

@opencv-alalek opencv-alalek May 23, 2025

Choose a reason for hiding this comment

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

License header.

Copy link
Copy Markdown
Contributor Author

@asmorkalov asmorkalov May 23, 2025

Choose a reason for hiding this comment

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

There are no license headers in IPP HAL for now. I'll fix it with another PR.

@asmorkalov asmorkalov self-assigned this May 24, 2025
@asmorkalov asmorkalov merged commit 0a5352e into opencv:4.x May 24, 2025
27 of 28 checks passed
@asmorkalov asmorkalov mentioned this pull request May 27, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@opencv-alalek opencv-alalek opencv-alalek left review comments

@mshabunin mshabunin Awaiting requested review from mshabunin

Assignees

@asmorkalov asmorkalov

Labels

category: core optimization

Projects

None yet

Milestone

4.12.0

Development

Successfully merging this pull request may close these issues.

2 participants

@asmorkalov @opencv-alalek