-
-
Notifications
You must be signed in to change notification settings - Fork 3.1k
feat: Add audio parameter support to gemini tts models #11287
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
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
- Add is_model_gemini_audio_model() method to detect TTS models - Include 'audio' parameter in supported params for TTS models - Map OpenAI audio parameter to Gemini speechConfig format - Add _extract_audio_response_from_parts() method to transform audio output to openai format
LiteLLM Mock Tests timed out after 8 minutes, all test were successful until time out. |
) | ||
|
||
# Map OpenAI audio parameter to Gemini speech config | ||
speech_config = {} |
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 we have these be typed dict's inside types/llms/vertex_ai.py
- so any future updates are also tracked correctly
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.
added typed dict for SpeechConfig and its child in types/llms/vertex_ai.py
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.
migrate test to test_litellm, and simplify tts model check
rest looks great. thank you for your work on this
- simplified gemini tts model detection - moved gemini_tts test to test_litellm
Thanks @AyrennC would you mind contributing docs for the change, so people know how to use this? For VertexAI - here Contributing guide - https://docs.litellm.ai/docs/extras/contributing (although it's just an .md change, so i'm sure you can just do it on github as well) |
Title
Add 'audio' params support to all gemini tts models
Relevant issues
Fixes #11250
Fixes #11118
Pre-Submission checklist
Please complete all items before asking a LiteLLM maintainer to review your PR
tests/litellm/
directory, Adding at least 1 test is a hard requirement - see detailsmake test-unit
Type
🆕 New Feature
🐛 Bug Fix
✅ Test
Changes