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
[Audio]: Add 'on_eos' event #8609
base: master
Are you sure you want to change the base?
Conversation
As well as unittests, this needs some documentation. I've read the code, and I am still just guessing what EOS stands for. |
Tests are ready. If somebody has any suggestions, let me know. Tests are not my thing. |
I'm not familiar with the audio providers so I have nothing to say about it, but you can use the |
Done. Thank you! |
Nice! You don't need the def test_on_eos_event(self, kivy_clock):
... |
It won't work. Take a look at this StackOverflow question and this code snippet. |
Sorry, I didn't know that. |
Seems like
|
This is not related with this PR. I think it should be clarified in a separate issue. |
Yes, I'll open an issue. (Sorry I forgot to mention that the issue occurs on the master branch) |
Good evening!
I want to add
on_eos
event to Kivy audio system. It works the same way as in the video, but is only available withffpyplayer
andgstplayer
providers.Related issue: #2156.
P.S.: Could someone please advise me on how the tests for this PR should look? Or is it not necessary to write them for this case?Maintainer merge checklist
Component: xxx
label.api-deprecation
orapi-break
label.release-highlight
label to be highlighted in release notes.versionadded
,versionchanged
as needed.