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

use OpenAI's TTS in place of the SpeechSynthesisUtterance API #5

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

Conversation

wobsoriano
Copy link

@wobsoriano wobsoriano commented Dec 12, 2023

This PR integrates OpenAI's TTS in place of the browser's SpeechSynthesisUtterance API.

Pros

  • Audio API has better browser support.
  • Natural-sounding audio.

Cons

  • Longer response times due to the additional fetch call.
  • Additional cost

@wobsoriano wobsoriano changed the title feat: replace SpeechSynthesisUtterance with OpenAI TTS API use OpenAI's TTS in place of the browser's SpeechSynthesisUtterance API Dec 12, 2023
@wobsoriano wobsoriano changed the title use OpenAI's TTS in place of the browser's SpeechSynthesisUtterance API use OpenAI's TTS in place of the SpeechSynthesisUtterance API Dec 12, 2023
@gregsadetsky
Copy link
Owner

hey, thank you for this! I think that perhaps we could have it as an option so that people could still use the browser built-in TTS if they wanted, but this option for better voices would be available as well.

I'm definitely concerned with adding any additional delays (i.e. api calls), as anything to get interactions as close as possible to real time is much more important in my opinion

thanks again

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.

2 participants