Skip to content

videoio(test): test skip due to non-updated FFmpeg wrapper#22933

Merged
opencv-pushbot merged 1 commit intoopencv:4.xfrom
alalek:fixup_22894
Dec 8, 2022
Merged

videoio(test): test skip due to non-updated FFmpeg wrapper#22933
opencv-pushbot merged 1 commit intoopencv:4.xfrom
alalek:fixup_22894

Conversation

@alalek
Copy link
Copy Markdown
Member

@alalek alalek commented Dec 8, 2022

relates #22894

Fail messages on Windows:

[ RUN      ] videoio_ffmpeg_16bit.basic/0, where GetParam() = ("FFV1", "avi", false, true)
[ INFO:0@44.452] global /build/opencv/modules/videoio/src/cap_interface.hpp (188) warnUnusedParameters VIDEOIO: unused parameter: [5]=2 / 0x0000000000000002
C:\build\precommit_windows64\4.x\opencv\modules\videoio\test\test_ffmpeg.cpp(595): error: Expected equality of these values:
  isSupported
    Which is: true
  writer.isOpened()
    Which is: false
[  FAILED  ] videoio_ffmpeg_16bit.basic/0, where GetParam() = ("FFV1", "avi", false, true) (0 ms)
[ RUN      ] videoio_ffmpeg_16bit.basic/1, where GetParam() = ("FFV1", "mkv", false, true)
[ INFO:0@44.453] global /build/opencv/modules/videoio/src/cap_interface.hpp (188) warnUnusedParameters VIDEOIO: unused parameter: [5]=2 / 0x0000000000000002
C:\build\precommit_windows64\4.x\opencv\modules\videoio\test\test_ffmpeg.cpp(595): error: Expected equality of these values:
  isSupported
    Which is: true
  writer.isOpened()
    Which is: false
[  FAILED  ] videoio_ffmpeg_16bit.basic/1, where GetParam() = ("FFV1", "mkv", false, true) (1 ms)

@mshabunin
Copy link
Copy Markdown
Contributor

Should I rebuild and update the plugin?

@alalek
Copy link
Copy Markdown
Member Author

alalek commented Dec 8, 2022

It will be done on the next week after version bump.

Copy link
Copy Markdown
Contributor

@mshabunin mshabunin 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!

@opencv-pushbot opencv-pushbot merged commit 93c4bca into opencv:4.x Dec 8, 2022
@alalek alalek mentioned this pull request Jan 8, 2023
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.

3 participants