-
-
Notifications
You must be signed in to change notification settings - Fork 3.8k
[UI] Add Deepgram provider to supported providers list and mappings #11634
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 ↗︎
|
|
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.
Pull Request Overview
Adds Deepgram as a supported provider by updating the enum, mapping, and logo entries.
- Introduce
Deepgram
inProviders
enum - Extend
provider_map
andproviderLogoMap
withDeepgram
Comments suppressed due to low confidence (3)
ui/litellm-dashboard/src/components/provider_info_helpers.tsx:55
- [nitpick] The constant name
provider_map
uses snake_case, which is inconsistent with the camelCase naming ofproviderLogoMap
. Consider renaming it toproviderMap
for consistency.
export const provider_map: Record<string, string> = {
ui/litellm-dashboard/src/components/provider_info_helpers.tsx:55
- [nitpick] Consider using
Record<Providers, string>
instead ofRecord<string, string>
forprovider_map
to leverage theProviders
enum and improve type safety.
export const provider_map: Record<string, string> = {
ui/litellm-dashboard/src/components/provider_info_helpers.tsx:60
- New provider
Deepgram
has been added, but there are no tests covering its mapping or logo retrieval. Please add unit tests to verifyDeepgram
appears correctly inprovider_map
andproviderLogoMap
.
Deepgram: "deepgram"
…erriAI#11634) * Add Deepgram provider to supported providers list and mappings * add logo * Add deepgram to model cost map * ui - require api key for deepgram * fix logo path --------- Co-authored-by: Cursor Agent <cursoragent@cursor.com>
…erriAI#11634) * Add Deepgram provider to supported providers list and mappings * add logo * Add deepgram to model cost map * ui - require api key for deepgram * fix logo path --------- Co-authored-by: Cursor Agent <cursoragent@cursor.com>
[UI] Add Deepgram provider to supported providers list and mappings
https://www.loom.com/share/70309ec0129c44d183f1f0c8c3c7629c?sid=a2d5aac0-6d67-4712-8a40-49e7dc9254c2
Relevant issues
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
✅ Test
Changes