-
Notifications
You must be signed in to change notification settings - Fork 212
Conversation
Codecov Report
@@ Coverage Diff @@
## master #726 +/- ##
==========================================
- Coverage 89.37% 89.36% -0.02%
==========================================
Files 198 198
Lines 10552 10565 +13
==========================================
+ Hits 9431 9441 +10
- Misses 1121 1124 +3
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
I have an strange error, while training on mp3 data everything works fine, but when using mp3 files to predict librosa is never called to load the audio. When using wav-files the print statement executes and everything works fine.
cause the audio never gets loaded an NoneType error returns |
Hey @flozi00 great work here! I fixed the MP3 predicting issue, just needed to add |
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.
Awesome, LGTM 😃
What does this PR do?
This PR replaces soundfile with librosa to support more audio formats like .mp3 by default
Fixes #724
Before submitting
PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃