-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Fix Audio.encode_example() when writing an array #3998
Fix Audio.encode_example() when writing an array #3998
Conversation
The documentation is not available anymore as the PR was closed or merged. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks a lot for the fix!
Do you think we could add some regression test on this issue?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks ! and thanks for the test as well :)
@albertvillanova do you think this line is enough? that's why we missed this bug, we didn't check this case |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great. Thank you!! :)
@@ -64,6 +64,7 @@ def test_audio_feature_type_to_arrow(): | |||
lambda audio_path: {"path": audio_path, "bytes": open(audio_path, "rb").read()}, | |||
lambda audio_path: {"path": None, "bytes": open(audio_path, "rb").read()}, | |||
lambda audio_path: {"bytes": open(audio_path, "rb").read()}, | |||
lambda audio_path: {"array": [0.1, 0.2, 0.3], "sampling_rate": 16_000}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good!
Closes #3996