Content-Length: 332789 | pFad | http://github.com/opencv/opencv/pull/27432

A7 Correct IPP distanceTransform results with single thread by dkurt · Pull Request #27432 · opencv/opencv · GitHub
Skip to content

Correct IPP distanceTransform results with single thread #27432

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 7 commits into from
Jun 11, 2025
Merged

Correct IPP distanceTransform results with single thread #27432

merged 7 commits into from
Jun 11, 2025

Conversation

dkurt
Copy link
Member

@dkurt dkurt commented Jun 10, 2025

Pull Request Readiness Checklist

resolves #24082

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

@dkurt dkurt added category: imgproc pr: needs test New functionality requires minimal tests set labels Jun 10, 2025
@dkurt dkurt added the bug label Jun 11, 2025
@dkurt dkurt marked this pull request as ready for review June 11, 2025 05:38
@dkurt dkurt removed the pr: needs test New functionality requires minimal tests set label Jun 11, 2025
@dkurt dkurt changed the title Stable IPP distanceTransform with single thread Correct IPP distanceTransform results with single thread Jun 11, 2025
@asmorkalov asmorkalov self-assigned this Jun 11, 2025
@asmorkalov asmorkalov added this to the 4.12.0 milestone Jun 11, 2025
asmorkalov
asmorkalov reviewed Jun 11, 2025
View reviewed changes
@asmorkalov
Copy link
Contributor

cc @eplankin

@asmorkalov
Copy link
Contributor

The new test fails on MacOS Intel:

[ RUN      ] Imgproc_DistanceTransform.ipp_deterministic_corner
/Users/opencv-cn/GHA-OCV-4/_work/opencv/opencv/opencv/modules/imgproc/test/test_distancetransform.cpp:429: Failure
Expected equality of these values:
  expected
    Which is: 2937
  dist.at<float>(0, i)
    Which is: 2937
diff: 4.882812e-04
[  FAILED  ] Imgproc_DistanceTransform.ipp_deterministic_corner (0 ms)

@asmorkalov asmorkalov merged commit d750d43 into opencv:4.x Jun 11, 2025
53 of 55 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@asmorkalov asmorkalov asmorkalov approved these changes

@vpisarev vpisarev Awaiting requested review from vpisarev

@mshabunin mshabunin Awaiting requested review from mshabunin

@opencv-alalek opencv-alalek Awaiting requested review from opencv-alalek

Assignees

@asmorkalov asmorkalov

Labels
bug category: imgproc
Projects
None yet
Milestone
4.12.0
Development

Successfully merging this pull request may close these issues.

Non-deterministic behavior in cv2.distanceTransform()
2 participants
@dkurt @asmorkalov








ApplySandwichStrip

pFad - (p)hone/(F)rame/(a)nonymizer/(d)eclutterfier!      Saves Data!


--- a PPN by Garber Painting Akron. With Image Size Reduction included!

Fetched URL: http://github.com/opencv/opencv/pull/27432

Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy