Skip to content
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

Add WAV header option for STT #10

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Add WAV header option for STT #10

wants to merge 2 commits into from

Conversation

x3a
Copy link
Contributor

@x3a x3a commented Oct 24, 2020

This adds support for sending raw 16-bit 16Khz mono audio to the STT endpoint of the Rhasspy HTTP API without a WAV header, thus providing a convenient way to send headerless WAV audio e.g. recorded with PyAudio.

@x3a x3a changed the title Add WAV header option for SST Add WAV header option for STT Oct 24, 2020
This adds support for sending raw 16-bit 16Khz mono audio to the STT
endpoint of the Rhasspy HTTP API without a WAV header, thus providing a
convenient way to send headerless WAV audio e.g. recorded with PyAudio.
@x3a
Copy link
Contributor Author

x3a commented Oct 24, 2020

Oh, I've just realized that this kind of functionality is already provided by stream_to_text defined in __init__.py, but maybe this small optional toggle flag I've added for the WAV header is more compact than a whole function for this, any feedback?

@koenvervloesem
Copy link
Member

@synesthesiam what do you think?

@synesthesiam
Copy link
Member

I think this is fine. There are cases where you might have RAW audio that you'd like to push as a whole.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants