-
Notifications
You must be signed in to change notification settings - Fork 714
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
Switch back to Cython and use scikit-build for Python module #271
Conversation
Use the SoundSwallower Cython code as a starting point
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
int ps_reinit_feat(ps_decoder_t *ps, cmd_ln_t *config) | ||
logmath_t *ps_get_logmath(ps_decoder_t *ps) | ||
int ps_start_stream(ps_decoder_t *ps) | ||
int ps_get_in_speech(ps_decoder_t *ps) |
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.
Can you expose get_in_speech as well? That would let you do some nice things like keep a mic open but discard the. audio up to x msec before speech starts
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.
ahh, funny you mention this, because I had removed get_in_speech as the internal VAD, which it used, was buggy and also not a very good VAD. do you have code that uses it already? I am trying to figure out what exactly the use case is for it and whether it can be supported or not...
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.
No -- and actually we can just check if there are word in the hyp, so I wouldn't worry about it
Cython makes somewhat larger binaries, but it has the following advantages:
The SWIG code is unmaintained since a long time, I don't have time to support it, and in any case the Ruby bindings (https://github.com/watsonbox/pocketsphinx-ruby) don't even use it.