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

Codestral tts #1302

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

Codestral tts #1302

wants to merge 6 commits into from

Conversation

Notnaton
Copy link
Collaborator

@Notnaton Notnaton commented Jun 11, 2024

Describe the changes you have made:

Added tts for codestral profile

Reference any relevant issues (e.g. "Fixes #000"):

Pre-Submission Checklist (optional but appreciated):

  • I have included relevant documentation updates (stored in /docs)
  • I have read docs/CONTRIBUTING.md
  • I have read docs/ROADMAP.md

OS Tests (optional but appreciated):

  • Tested on Windows
  • Tested on MacOS
  • Tested on Linux

@MikeBirdTech
Copy link
Collaborator

MikeBirdTech commented Jun 11, 2024

Thanks Anton!!

Video of profile in action: https://www.youtube.com/watch?v=l9QKGc4AL60

@KillianLucas
Copy link
Collaborator

NICE!! Love the idea of a local voice assistant. I do think we should keep OI's main repo focused on text, but a profile is a great place to add this extension.

Could the other new files perhaps be pushed into the profile? For example, I think we shouldn't have everyone installing those dependencies, maybe just do a try except around the imports of them, then if that fails, you could print a message saying "you need to pip install these dependencies". Then I think the live_transcribe could also go in the profile, at the top. What do you think?

@Notnaton
Copy link
Collaborator Author

Notnaton commented Jun 15, 2024

Could the other new files perhaps be pushed into the profile?

Maybe profiles/utils/ is a good spot for glue needed for awesome profiles

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