feat:Updated Deep Infra models #175
Conversation
WalkthroughThis pull request updates two core files by modifying the available model identifiers. The changes involve adding new models with detailed metadata and removing several outdated ones. Both the enumeration in Changes
Possibly related PRs
Suggested reviewers
Poem
✨ Finishing Touches
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (3)
src/DeepInfra/src/DeepInfraModelIds.cs (3)
493-502: Consider adding a description for the Gemini-1.5-flash model.While the model addition is correct with proper metadata and pricing information, the description field is empty. Consider adding a brief description of the Gemini-1.5-flash model's capabilities and features for consistency with other model entries.
504-513: Consider adding a description for the Gemini-1.5-flash-8b model.The model entry is complete with appropriate metadata, but it would benefit from a description of the model's capabilities and use cases, similar to other model entries in this enum.
515-524: Consider adding a description for the Gemini-2.0-flash-001 model.The model entry includes proper metadata but lacks a description. Adding information about this model's features would improve consistency with other entries and provide better documentation for developers.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
src/DeepInfra/src/DeepInfraModelIds.cs(3 hunks)src/DeepInfra/src/DeepInfraModelProvider.cs(3 hunks)
⏰ Context from checks skipped due to timeout of 90000ms (1)
- GitHub Check: Build and test / Build, test and publish
🔇 Additional comments (6)
src/DeepInfra/src/DeepInfraModelIds.cs (3)
31-40: New model addition looks good.The addition of the
Qwq32Bmodel is well-documented with complete metadata including context length, pricing information, and a comprehensive description detailing its capabilities.
42-51: Phi4MultimodalInstruct model is well-documented.The addition of Microsoft's Phi-4-multimodal-instruct model is properly implemented with detailed metadata. The documentation clearly describes its multimodal capabilities (text, image, audio inputs) and supported languages.
823-832: Well-documented multilingual embedding model addition.The addition of
BgeM3Multiis well implemented with comprehensive documentation that clearly describes the model's multilingual capabilities and retrieval functionalities.src/DeepInfra/src/DeepInfraModelProvider.cs (3)
14-15: New model entries are consistent with the enum definition.The additions of
Qwq32BandPhi4MultimodalInstructto the Models dictionary are correctly implemented with appropriate metadata parameters that match their enum descriptions.
56-58: Gemini models properly added with very high context lengths.The three Gemini models have been correctly added to the Models dictionary. Note the exceptionally high context length (1,000,000 tokens) for these models, which represents a significant capability compared to most other models in the collection.
86-86: BgeM3Multi embedding model correctly added.The BgeM3Multi model has been properly added to the dictionary with the correct pricing structure for embedding models (input cost but no output cost).
Created by Github Actions
Summary by CodeRabbit
New Features
Chores