Skip to content

Test cv::cuda::event#3144

Merged
alalek merged 14 commits intoopencv:4.xfrom
cudawarped:test_cuda_event
Jan 11, 2022
Merged

Test cv::cuda::event#3144
alalek merged 14 commits intoopencv:4.xfrom
cudawarped:test_cuda_event

Conversation

@cudawarped
Copy link
Copy Markdown
Contributor

@cudawarped cudawarped commented Dec 31, 2021 •
edited by alalek
Loading

Add tests to cv::cuda::event. Some of these tests depend on opencv/opencv#21374 because they create events using combinations of flags.

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 other license that is incompatible with OpenCV
  • The PR is proposed to proper branch
  • There is reference to 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
opencv=fix_cuda_event_flags
force_builders=Custom
buildworker:Custom=linux-4,linux-6
Xbuild_image:Custom=ubuntu-cuda:18.04
build_image:Custom=ubuntu-cuda-cc52:18.04

James Bowley and others added 8 commits June 11, 2019 09:57
Add missing codecs to cudacodec which uses Nvidia Video Codec SDK inc…
736fe2a 
…luding checks to ensure codec used in input video file is supported on the current device.
Merge branch 'master' of https://github.com/opencv/opencv_contrib int…
4d4fc5d 
…o update_nvcuvid_codecs
Merge branch 'master' of https://github.com/cudawarped/opencv_contrib
3b5acef 
# Conflicts:
#	modules/cudacodec/include/opencv2/cudacodec.hpp
#	modules/cudacodec/src/video_decoder.cpp
@cudawarped
Merge branch 'master' of https://github.com/opencv/opencv_contrib
511c8a5
@cudawarped
Merge branch 'master' of https://github.com/opencv/opencv_contrib
20505c0
@cudawarped
Merge branch 'master' of https://github.com/opencv/opencv_contrib
f57f140
@cudawarped
Merge branch 'master' of https://github.com/opencv/opencv_contrib
aa6a869
@cudawarped
Merge branch 'master' of https://github.com/opencv/opencv_contrib
a59e0a2
@cudawarped cudawarped mentioned this pull request Dec 31, 2021
Merged
6 tasks
@alalek
Copy link
Copy Markdown
Member

alalek commented Jan 10, 2022

Please take a look on these new warnigs:

/build/precommit-contrib_custom_linux64/4.x/opencv_contrib/modules/cudaarithm/test/test_event.cpp:116:23: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
/build/precommit-contrib_custom_linux64/4.x/opencv_contrib/modules/cudaarithm/test/test_event.cpp:127:16: warning: unused variable 'elTimeMs' [-Wunused-variable]

alalek
alalek reviewed Jan 11, 2022
View reviewed changes
modules/cudaarithm/test/test_event.cpp Outdated
@@ -0,0 +1,142 @@
/*M///////////////////////////////////////////////////////////////////////////////////////
Copy link
Copy Markdown
Member

@alalek alalek Jan 11, 2022

Choose a reason for hiding this comment

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

Please use short license header: https://github.com/opencv/opencv/wiki/Coding_Style_Guide#file-structure

// This file is part of OpenCV project.
// It is subject to the license terms in the LICENSE file found in the top-level directory
// of this distribution and at http://opencv.org/license.html.

alalek
alalek approved these changes Jan 11, 2022
View reviewed changes
Copy link
Copy Markdown
Member

@alalek alalek left a comment

Choose a reason for hiding this comment

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

Thank you 👍

@alalek alalek merged commit fdb53d6 into opencv:4.x Jan 11, 2022
@alalek alalek mentioned this pull request Feb 22, 2022
Merged
@cudawarped cudawarped mentioned this pull request Aug 19, 2022
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alalek alalek alalek approved these changes

Assignees

@alalek alalek

Labels

category: cuda test

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@cudawarped @alalek