-
-
Notifications
You must be signed in to change notification settings - Fork 2
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
Test #23
Test #23
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## dev #23 +/- ##
=======================================
Coverage ? 100.00%
=======================================
Files ? 3
Lines ? 56
Branches ? 4
=======================================
Hits ? 56
Misses ? 0
Partials ? 0 ☔ View full report in Codecov by Sentry. |
One of the tests is failing (Ubuntu). It suggests using |
""" | ||
>>> import os | ||
>>> from nava import play | ||
>>> play(os.path.join("others", "test.wav")) |
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.
I couldn't find the exact place in which we test the correct performance of the play
function. (As you said, comparing waveforms and etc.)
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.
Github Action does not install any drivers by default. After installing the driver, if the system succeeds in playing the file, no error will occur. That's enough at the moment.
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.
LGTM.
Also I trimmed the test.wav
down to 60.3KB
using sox
. Hope everything is right, @sepandhaghighi please check the sound for double-check.
This is the current sound's spectrogram (drawn by sciencemusic):
LGTM 🔥 |
Reference Issues/PRs
#18
What does this implement/fix? Explain your changes.
Any other comments?