Skip to content

add missing include#24167

Merged
opencv-alalek merged 2 commits intoopencv:4.xfrom
autoantwort:missing-include
Aug 17, 2023
Merged

add missing include#24167
opencv-alalek merged 2 commits intoopencv:4.xfrom
autoantwort:missing-include

Conversation

@autoantwort
Copy link
Copy Markdown
Contributor

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 ([opencv4[halide]] Build failure microsoft/vcpkg#33200)

@dkurt
Copy link
Copy Markdown
Member

dkurt commented Aug 17, 2023

Just curious, do you use Halide backend or this was found by some static analysis tool? Halide going to be removed in #24123

@autoantwort
Copy link
Copy Markdown
Contributor Author

Just curious, do you use Halide backend or this was found by some static analysis tool?

More or less static analysis tool. I tested all features of the opencv port in vcpkg.

@opencv-alalek opencv-alalek self-assigned this Aug 17, 2023
@opencv-alalek opencv-alalek merged commit f5a1453 into opencv:4.x Aug 17, 2023
@asmorkalov asmorkalov mentioned this pull request Sep 11, 2023
@opencv-alalek opencv-alalek added the port/backport done Label for maintainers. Authors of PR can ignore this label Oct 31, 2023
thewoz pushed a commit to thewoz/opencv that referenced this pull request Jan 4, 2024
thewoz pushed a commit to thewoz/opencv that referenced this pull request May 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug category: build/install category: dnn port/backport done Label for maintainers. Authors of PR can ignore this

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants