Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add MP3 support (with minimp3) for Audio Dataset and IOTensor #801

Merged
merged 2 commits into from
Feb 21, 2020

Conversation

yongtang
Copy link
Member

This PR adds MP3 support (with minimp3) for Audio Dataset and IOTensor.
The difference between this PR and FFmpeg is that FFmpeg is only supported
on specific Ubuntu 16.04 and 18.04, though external .so.
This PR builds minimp3 as part of the program and support Windows/Linux/macOS.

/cc @faroit in case you might be interested.

Signed-off-by: Yong Tang yong.tang.github@outlook.com

This PR adds MP3 support (with minimp3) for Audio Dataset and IOTensor.
The difference between this PR and FFmpeg is that FFmpeg is only supported
on specific Ubuntu 16.04 and 18.04, though external .so.
This PR builds minimp3 as part of the program and support Windows/Linux/macOS.

Signed-off-by: Yong Tang <yong.tang.github@outlook.com>
Signed-off-by: Yong Tang <yong.tang.github@outlook.com>
Copy link
Member

@terrytangyuan terrytangyuan left a comment

Choose a reason for hiding this comment

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

Awesome. Looks great!

@terrytangyuan terrytangyuan merged commit c2e67f8 into tensorflow:master Feb 21, 2020
@yongtang yongtang deleted the mp3 branch February 21, 2020 03:03
@lieff
Copy link

lieff commented Feb 24, 2020

The error of skipping the not decodable first frames in callback mode was fixed here lieff/minimp3@2b9a023 . It's rare case, but may be worth updating.

@yongtang
Copy link
Member Author

Thanks @lieff for the update! I have created a PR in #807 to bring minimp3 to 2b9a023.

By the way, thanks for your great work for minimp3 👍 !

@lieff
Copy link

lieff commented Feb 24, 2020

Thanks. Glad to help :)

@faroit faroit mentioned this pull request Mar 9, 2020
11 tasks
@faroit
Copy link

faroit commented Mar 9, 2020

@yongtang sorry, didn't had much time recently. I update my benchmarks to include 0.12.0 and tested the minimp3 performance.

Its almost factor 10 faster than ffmpeg! 🐎

benchmark_tf

@yongtang
Copy link
Member Author

yongtang commented Mar 9, 2020

@faroit Nice 👍 !

i-ony pushed a commit to i-ony/io that referenced this pull request Feb 8, 2021
…flow#801)

* Add MP3 support (with minimp3) for Audio Dataset and IOTensor

This PR adds MP3 support (with minimp3) for Audio Dataset and IOTensor.
The difference between this PR and FFmpeg is that FFmpeg is only supported
on specific Ubuntu 16.04 and 18.04, though external .so.
This PR builds minimp3 as part of the program and support Windows/Linux/macOS.

Signed-off-by: Yong Tang <yong.tang.github@outlook.com>

* Update minimp3 to carry patch on Windows

Signed-off-by: Yong Tang <yong.tang.github@outlook.com>
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.

4 participants