-
Notifications
You must be signed in to change notification settings - Fork 72
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
Minor Kaldi updates #302
Minor Kaldi updates #302
Conversation
This looks good to me. What are the spurious disconnect messages you mentioned? |
I mentioned in another thread Kaldi with Linux and possibly Windows error messages on disconnect related to the audio library. I've been quite busy with work so I haven't had a chance to test this yet. I can post examples tomorrow or so. |
Previously, depending on timing during disconnect, the audio thread could raise an error. Now, the disconnect thread will wait and join with the audio thread being closed. Actually, I should probably add a time out to the join call. If the audio thread is broken some how, we would probably rather hurry up and disconnect, and accept any later error messages. @LexiconCode No rush! |
Here are the error messages for Linux Using your latest daanzu:kaldi dragonfly branch.
Using standard dragonfly 0.27.1
|
@LexiconCode And this is when you just call disconnect, right? |
@daanzu hmmm. Test Rule without Caster. Test with
However there seems to be issues calling Popen on Linux after disconnect. This problem is semi-intermittent with subprocess.Popen.
semi-intermittent
Sometimes initializes twice with
The last bit is keyboard interrupt |
I'll merge this now for dragonfly2 release version 0.28.0. I have verified that the issues I had running the tests before are now resolved. I have also simplified the Kaldi engine test file ( Regarding the above errors, perhaps these should be documented in a separate issue? |
@Danesprite Merge sounds good! I am planning on working on fixing the audio, but putting it in an issue doesn't hurt. |
@LexiconCode Curious if this reduces spurious messages with disconnect.