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

Change to float32 for AudioIOTensor and AudioIODataset in case of MP3 and Ogg #859

Merged
merged 2 commits into from
Mar 17, 2020

Conversation

yongtang
Copy link
Member

Change to float32 for AudioIOTensor and AudioIODataset in case of MP3 and Ogg, and make related changes to decode_ogg.

This is related to discussion to #839 (comment)

Note decode_mp3 is not in place yet. I will defer to #815 for now. May pick up if needed.

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

Signed-off-by: Yong Tang <yong.tang.github@outlook.com>
Signed-off-by: Yong Tang <yong.tang.github@outlook.com>
@yongtang yongtang merged commit 5a54e6f into tensorflow:master Mar 17, 2020
@yongtang yongtang deleted the ogg branch March 17, 2020 15:23
i-ony pushed a commit to i-ony/io that referenced this pull request Feb 8, 2021
… and Ogg (tensorflow#859)

* Use float for ogg as ogg is lossy anyway

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

* Change to float32 for AudioIOTensor and AudioIODataset in case of MP3

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.

2 participants