Add new AI models, update docs - #137
Conversation
WalkthroughThe pull request updates the LiteLLM configuration by adding new Anthropic models, modifying Openrouter model entries, and explicitly grouping local Ollama models. It also enhances documentation for open source models by adding resources for limited GPU environments and updating notes on coding-optimized models. Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant LiteLLM Config
participant Anthropic API
participant Openrouter API
participant Local Model
User->>LiteLLM Config: Request model inference
alt Anthropic model
LiteLLM Config->>Anthropic API: Forward request with API key
Anthropic API-->>LiteLLM Config: Return response
else Openrouter model
LiteLLM Config->>Openrouter API: Forward request with API key
Openrouter API-->>LiteLLM Config: Return response
else Local model
LiteLLM Config->>Local Model: Invoke local inference
Local Model-->>LiteLLM Config: Return response
end
LiteLLM Config-->>User: Return inference result
Possibly related PRs
Note ⚡️ AI Code Reviews for VS Code, Cursor, WindsurfCodeRabbit now has a plugin for VS Code, Cursor and Windsurf. This brings AI code reviews directly in the code editor. Each commit is reviewed immediately, finding bugs before the PR is raised. Seamless context handoff to your AI code agent ensures that you can easily incorporate review feedback. 📜 Recent review detailsConfiguration used: .coderabbit.yaml 📒 Files selected for processing (2)
🚧 Files skipped from review as they are similar to previous changes (2)
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:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. 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
Documentation and Community
|
There was a problem hiding this comment.
Actionable comments posted: 2
🧹 Nitpick comments (5)
docs/ai/open-models.md (3)
12-14: Use a proper Markdown heading for the new section
The “Tools for limited GPU resources” block is inserted inline—consider converting it into a level-3 heading (### Tools for limited GPU resources) to match the surrounding structure and improve readability and navigation.
16-16: Convert the TODO into a tracked task
The inlineTODO: Update the list of best modelsshould either be resolved before merging or linked to a GitHub issue so it doesn’t get lost.
28-28: Format the coding-optimized note as an admonition
Instead of a plain sentence, consider using a Markdown quote block (> Note:) or an admonition to highlight that proprietary models now excel at coding. This will make the note more visible.docker/ai/litellm/config.yaml (2)
24-38: Validate new Anthropic model entries and API key syntax
The new entries forclaude-3-7-sonnet,claude-opus-4-0, andclaude-sonnet-4-0look structurally sound, but please:
- Confirm the
modelstrings (e.g.,anthropic/claude-3-7-sonnet-latest) exactly match Anthropic’s published names.- Ensure
"os.environ/ANTHROPIC_API_KEY"is correctly interpreted by Litellm (it may need alternate placeholder syntax or removal of quotes).
78-93: Review local model configuration and environment variables
The local models (ollama-local-phiandollama-mac-mistral) are useful additions, but:
- Double-check that
LOCAL_OLLAMA_API_BASEvsREMOTE_OLLAMA_API_BASEalign with your deployment and are documented in your README or environment setup.- Consider renaming
ollama-mac-mistralto something likeollama-mistral-localfor clarity and consistency.
📜 Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
docker/ai/litellm/config.yaml(3 hunks)docs/ai/open-models.md(2 hunks)
🔇 Additional comments (1)
docker/ai/litellm/config.yaml (1)
43-47: Verify Openrouter model IDs and key handling
The updatedgpt-4oentry and newo4-mini-highmodel should be cross-checked against Openrouter’s current naming conventions. Also confirm that the"os.environ/OPENROUTER_API_KEY"literal is parsed as an environment variable by Litellm’s loader.Also applies to: 54-59
Summary by CodeRabbit
New Features
Documentation