Skip to content

Conversation

@yongtang
Copy link
Member

This PR does two things:

  1. Add ffmpeg support for Ubuntu 20.04
    In the past only Ubuntu 16.04 and 18.04 are supported for ffmpeg audio/video Dataset.
    In this PR ffmpeg support for Ubuntu 20.04 has been added as Ubuntu 20.04 has been released this year.
    Also fix Invalid return value 0 for stream protocol issue when LIBAVFORMAT_VERSION_MAJOR > 57

  2. Switch to python 3.8 for tests
    In the past we use python 3.6/3.7 as the default version for testing as some packages for testing are not available for 3.8 yet, even though many python systems have been upgraded to 3.8. For example, Ubuntu 20.04 and macOS's default system pythons are already 3.8.
    This PR switches to use python 3.8 for testings. This should also reduce GitHub Actions jobs a little bit.
    This PR also updates ffmpeg video samples (as there is a small different between Ubuntu 18.04 and Ubuntu 20.04 ffmpeg)

Signed-off-by: Yong Tang [email protected]

In the past only Ubuntu 16.04 and 18.04 are supported for ffmpeg audio/video
Dataset.

In this PR ffmpeg support for Ubuntu 20.04 has been added as Ubuntu 20.04 has
been released this year.

Also fix `Invalid return value 0 for stream protocol` issue when LIBAVFORMAT_VERSION_MAJOR > 57

Signed-off-by: Yong Tang <[email protected]>
@yongtang yongtang force-pushed the ffmpeg-4.2 branch 2 times, most recently from b4b0c22 to 8ff30df Compare November 16, 2020 02:37
strategy:
matrix:
python: ['3.6', '3.7']
python: ['3.8']
Copy link
Member

Choose a reason for hiding this comment

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

@yongtang maybe we can test with 3.7 and 3.8? Just to be sure of the compatibility as users might still be using 3.7.

In the past we use python 3.6/3.7 as the default version for testing
as some packages for testing are not available for 3.8 yet,
even though many python systems have been upgraded to 3.8. For example,
Ubuntu 20.04 and macOS's default system pythons are already 3.8.

This PR switches to use python 3.8 for testings. This should also reduce
GitHub Actions jobs a little bit.

This PR also updates ffmpeg video samples (as there is a small different between
Ubuntu 18.04 and Ubuntu 20.04 ffmpeg)

Signed-off-by: Yong Tang <[email protected]>
@terrytangyuan terrytangyuan merged commit a379390 into tensorflow:master Nov 16, 2020
@yongtang yongtang deleted the ffmpeg-4.2 branch November 16, 2020 14:00
i-ony pushed a commit to i-ony/io that referenced this pull request Feb 8, 2021
tensorflow#1195)

* Add ffmpeg support for Ubuntu 20.04

In the past only Ubuntu 16.04 and 18.04 are supported for ffmpeg audio/video
Dataset.

In this PR ffmpeg support for Ubuntu 20.04 has been added as Ubuntu 20.04 has
been released this year.

Also fix `Invalid return value 0 for stream protocol` issue when LIBAVFORMAT_VERSION_MAJOR > 57

Signed-off-by: Yong Tang <[email protected]>

* Switch to python 3.8 for tests

In the past we use python 3.6/3.7 as the default version for testing
as some packages for testing are not available for 3.8 yet,
even though many python systems have been upgraded to 3.8. For example,
Ubuntu 20.04 and macOS's default system pythons are already 3.8.

This PR switches to use python 3.8 for testings. This should also reduce
GitHub Actions jobs a little bit.

This PR also updates ffmpeg video samples (as there is a small different between
Ubuntu 18.04 and Ubuntu 20.04 ffmpeg)

Signed-off-by: Yong Tang <[email protected]>

* Add python3.7 test back, based on review comment

Signed-off-by: Yong Tang <[email protected]>
zheolong pushed a commit to zheolong/io-1 that referenced this pull request Jul 24, 2025
tensorflow#1195)

* Add ffmpeg support for Ubuntu 20.04

In the past only Ubuntu 16.04 and 18.04 are supported for ffmpeg audio/video
Dataset.

In this PR ffmpeg support for Ubuntu 20.04 has been added as Ubuntu 20.04 has
been released this year.

Also fix `Invalid return value 0 for stream protocol` issue when LIBAVFORMAT_VERSION_MAJOR > 57

Signed-off-by: Yong Tang <[email protected]>

* Switch to python 3.8 for tests

In the past we use python 3.6/3.7 as the default version for testing
as some packages for testing are not available for 3.8 yet,
even though many python systems have been upgraded to 3.8. For example,
Ubuntu 20.04 and macOS's default system pythons are already 3.8.

This PR switches to use python 3.8 for testings. This should also reduce
GitHub Actions jobs a little bit.

This PR also updates ffmpeg video samples (as there is a small different between
Ubuntu 18.04 and Ubuntu 20.04 ffmpeg)

Signed-off-by: Yong Tang <[email protected]>

* Add python3.7 test back, based on review comment

Signed-off-by: Yong Tang <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants