Skip to content

Fix image loading in tutorials code#23442

Merged
asmorkalov merged 3 commits intoopencv:4.xfrom
LaurentBerger:tuto_findFile
Apr 6, 2023
Merged

Fix image loading in tutorials code#23442
asmorkalov merged 3 commits intoopencv:4.xfrom
LaurentBerger:tuto_findFile

Conversation

@LaurentBerger
Copy link
Copy Markdown
Contributor

@LaurentBerger LaurentBerger commented Mar 31, 2023

Fixes #23378

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 self-requested a review April 1, 2023 11:51
return -1;
}
CommandLineParser parser( argc, argv, keys );
samples::addSamplesDataSearchSubDirectory( "./doc/tutorials/imgproc/histograms/template_matching/images" );
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please remove ./ at the beginning. Here and in the following places.

@asmorkalov asmorkalov self-assigned this Apr 3, 2023
@asmorkalov
Copy link
Copy Markdown
Contributor

Tried manually with Ubuntu 20.04.Works well for me. Thanks a lot for the contribution!

@LaurentBerger
Copy link
Copy Markdown
Contributor Author

Tried manually with Ubuntu 20.04.Works well for me. Thanks a lot for the contribution!

On windows
example_tutorial_out_of_focus_deblur_filter.exe -R=52 -SNR=5200
image

and with example_tutorial_out_of_focus_deblur_filter.exe -R=5 -SNR=100

image

@asmorkalov asmorkalov changed the title Issue 23378 Fix image loading in tutorials code Apr 4, 2023
Copy link
Copy Markdown
Contributor

@asmorkalov asmorkalov left a comment

Choose a reason for hiding this comment

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

👍

@asmorkalov asmorkalov merged commit 9742c73 into opencv:4.x Apr 6, 2023
@asmorkalov asmorkalov added this to the 4.8.0 milestone Apr 6, 2023
@LaurentBerger LaurentBerger deleted the tuto_findFile branch April 6, 2023 10:03
@asmorkalov asmorkalov mentioned this pull request May 31, 2023
@asmorkalov asmorkalov mentioned this pull request Jun 27, 2023
6 tasks
thewoz pushed a commit to thewoz/opencv that referenced this pull request Jan 4, 2024
Fix image loading in tutorials code opencv#23442

Fixes opencv#23378

### 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
- [X] The PR is proposed to the proper branch
- [X] There is a reference to the original bug report and related work
- [X] There is accuracy test, performance test and test data in opencv_extra repository, if applicable
      Patch to opencv_extra has the same branch name.
- [X] The feature is well documented and sample code can be built with the project CMake
thewoz pushed a commit to thewoz/opencv that referenced this pull request May 29, 2024
Fix image loading in tutorials code opencv#23442

Fixes opencv#23378

### 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
- [X] The PR is proposed to the proper branch
- [X] There is a reference to the original bug report and related work
- [X] There is accuracy test, performance test and test data in opencv_extra repository, if applicable
      Patch to opencv_extra has the same branch name.
- [X] 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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Load Image in tutorials

2 participants