θΏ™ζ˜―indexlocζδΎ›ηš„ζœεŠ‘οΌŒδΈθ¦θΎ“ε…₯任何密码
Skip to content

Conversation

@phazei
Copy link
Contributor

@phazei phazei commented Oct 14, 2024

Pull Request Type

  • ✨ feat
  • πŸ› fix
  • ♻️ refactor
  • πŸ’„ style
  • πŸ”¨ chore
  • πŸ“ docs

Relevant Issues

resolves #xxx

What is in this change?

This adds an input field for a custom BaseURL for the OpenAI TTS endpoint when selecting OpenAI as a TTS provider.

Since OpenAI has become an industry standard many tools have taken to mirroring their API for ease of integration.

With this change it allows for a much richer choice of options to use as a TTS provider.

I tested it using AllTalk_TTS's OpenAI speech endpoint and it works wonderfully.

Developer Validations

  • I ran yarn lint from the root of the repo & committed changes

  • [ x] Relevant documentation has been updated
    couldn't find any relevant docs, but it is notated in the env comments

  • [ x] I have tested my code functionality

  • [ x] Docker build succeeds locally
    In order to debug this, I needed to get the frontend/server/collector all working since the docker takes a VERY long time on the production-build 3/3 chown step. There has got to be a way to speed that up. Also, when running the servers independently, it doesn't save the config in any .env file so with every restart I had to set it up again, don't know what I missed there. I was also running it under WSL ubuntu, so that was it's own pita, my dev machine is a MBP, but only my PC can run the models :/

@phazei phazei changed the title Update OpenAI TTS config to allow a custom BaseURL Update OpenAI TTS config to allow a custom BaseURL allowing for any TTS engine with a compatible API Oct 14, 2024
@phazei phazei force-pushed the tts-openAI-compatible-endpoints branch from 639788f to 99af124 Compare October 14, 2024 00:52
@phazei phazei force-pushed the tts-openAI-compatible-endpoints branch from 99af124 to bdf3fb8 Compare October 14, 2024 00:54
@timothycarambat timothycarambat self-assigned this Oct 14, 2024
@timothycarambat timothycarambat added the PR:needs review Needs review by core team label Oct 14, 2024
@timothycarambat
Copy link
Member

Moved to #2487 (youll still have PR credit)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR:needs review Needs review by core team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants