Skip to content

videoio: print test params instead of indexes#26948

Merged
asmorkalov merged 2 commits intoopencv:4.xfrom
mshabunin:fix-videoio-test-params
Feb 26, 2025
Merged

videoio: print test params instead of indexes#26948
asmorkalov merged 2 commits intoopencv:4.xfrom
mshabunin:fix-videoio-test-params

Conversation

@mshabunin
Copy link
Copy Markdown
Contributor

videoio test names changed - use string instead of index.
E.g. videoio_read.threads/0 is now videoio_read.threads/h264_0_RAW.
It allows to filter tests independently of the platform.

Notes:

  • not all tests has been updated - only simpler ones and those which have varying parameters depending on platform

@mshabunin mshabunin force-pushed the fix-videoio-test-params branch from 9464468 to 67ffb32 Compare February 20, 2025 15:10
@asmorkalov asmorkalov self-assigned this Feb 20, 2025
@asmorkalov asmorkalov added this to the 4.12.0 milestone Feb 20, 2025
@mshabunin
Copy link
Copy Markdown
Contributor Author

Precommit tests passed twice with modified pipeline (opencv/ci-gha-workflow#225). I'll revert pipeline branch here. This PR should be merged together with pipeline changes. I'll port both to 5.x manually after that.

@asmorkalov asmorkalov merged commit 43551b7 into opencv:4.x Feb 26, 2025
25 of 28 checks passed
@mshabunin mshabunin deleted the fix-videoio-test-params branch February 26, 2025 11:05
asmorkalov pushed a commit that referenced this pull request Mar 1, 2025
videoio: print test params instead of indexes (5.x) #26987

Port of #26948 to 5.x
@asmorkalov asmorkalov mentioned this pull request Mar 4, 2025
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