Skip to content

Fixed the simd bugs of iPow8u and iPow16u#26061

Merged
asmorkalov merged 2 commits intoopencv:4.xfrom
penghuiho:fix-pow-bug
Aug 23, 2024
Merged

Fixed the simd bugs of iPow8u and iPow16u#26061
asmorkalov merged 2 commits intoopencv:4.xfrom
penghuiho:fix-pow-bug

Conversation

@penghuiho
Copy link
Copy Markdown
Contributor

Add the following cases in opencv_perf_core:

  • OCL_PowFixture_iPow.iPow/0, where GetParam() = (640x480, 8UC1)
  • OCL_PowFixture_iPow.iPow/2, where GetParam() = (640x480, 16UC1)

iPow8u and iPow16u failed to call to simd accelerating while executing.

Fix the bug by changing the input type of iPow_SIMD function.

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

Add the following case in opencv_perf_core:

* OCL_PowFixture_iPow.iPow/0, where GetParam() = (640x480, 8UC1)
* OCL_PowFixture_iPow.iPow/2, where GetParam() = (640x480, 16UC1)

iPow8u and iPow16u failed to call to simd accelerating while executing.

Fix the bug by changing the input type of iPow_SIMD function.
@asmorkalov asmorkalov self-requested a review August 23, 2024 11:33
@asmorkalov asmorkalov self-assigned this Aug 23, 2024
@asmorkalov
Copy link
Copy Markdown
Contributor

@penghuiho Thanks a lot for the contribution! The 4.x version fails back to Dummy function at

struct iPow_SIMD
. We always get serial code instead of vectorized. But the new test highlights couple of issues related to OpenCL code. I'll fix it separately.

@asmorkalov asmorkalov merged commit f4c2e4f into opencv:4.x Aug 23, 2024
thewoz pushed a commit to CobbsLab/OPENCV that referenced this pull request Feb 13, 2025
Fixed the simd bugs of iPow8u and iPow16u opencv#26061

Add the following cases in opencv_perf_core:

* OCL_PowFixture_iPow.iPow/0, where GetParam() = (640x480, 8UC1)
* OCL_PowFixture_iPow.iPow/2, where GetParam() = (640x480, 16UC1)

iPow8u and iPow16u failed to call to simd accelerating while executing.

Fix the bug by changing the input type of iPow_SIMD function.

### 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
- [ ] 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
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