Fix au.anthropic.claude opus 4 6 v1#20731
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Greptile OverviewGreptile SummaryThis PR correctly fixes the regional prefix for the Claude Opus 4.6 model in the Australia region, changing from the incorrect Key Changes:
Important Note: Confidence Score: 4/5
|
| Filename | Overview |
|---|---|
| model_prices_and_context_window.json | Correctly removes duplicate apac.anthropic.claude-opus-4-6-v1 entries and adds the proper au.anthropic.claude-opus-4-6-v1 model configuration for Australia region |
| litellm/model_prices_and_context_window_backup.json | Backup file correctly mirrors the changes made to the main configuration file |
| tests/test_litellm/test_claude_opus_4_6_config.py | Adds comprehensive test coverage for au. prefix and verifies the incorrect apac. prefix is not used, with detailed documentation of AWS regional prefix conventions |
Sequence Diagram
sequenceDiagram
participant User
participant LiteLLM
participant ModelConfig as model_prices_and_context_window.json
participant BedrockConverse as bedrock_converse_models
participant Bedrock as AWS Bedrock API
Note over ModelConfig: Before PR: Had duplicate apac. entries
Note over ModelConfig: After PR: Single au. entry (correct)
User->>LiteLLM: Request with au.anthropic.claude-opus-4-6-v1
LiteLLM->>ModelConfig: Load model configuration
ModelConfig-->>LiteLLM: Return au. model config
LiteLLM->>BedrockConverse: Register au. model
BedrockConverse-->>LiteLLM: Model registered
LiteLLM->>Bedrock: Call Australia region (ap-southeast-2)
Bedrock-->>LiteLLM: Response
LiteLLM-->>User: Return result
Note over ModelConfig,BedrockConverse: PR correctly maps Australia to au. prefix
Note over ModelConfig,BedrockConverse: apac. is for Singapore (ap-southeast-1)
Additional Comments (1)
|
|
@anttttti can you fix the merge conflicts, so we can merge this in? cc: @shin-bot-litellm for monitoring this pr |
…aude-opus-4-6-v1 # Conflicts: # litellm/model_prices_and_context_window_backup.json # model_prices_and_context_window.json
|
@krrishdholakia Merged. Easy conflicts. Updated changes from both side. |
Can this be merged? |
|
@krrishdholakia bump on this. Merge? |
|
done, thanks for the bump |
Relevant issues
Fixes missing au.anthropic.claude-opus-4-6-v1. Removed duplicate entries for apac.anthropic.claude-opus-4-6-v1, which doesn't exist
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-unitCI (LiteLLM team)
Branch creation CI run
Link:
CI run for the last commit
Link:
Merge / cherry-pick CI run
Links:
Type
🆕 New Feature
🐛 Bug Fix
🧹 Refactoring
📖 Documentation
🚄 Infrastructure
✅ Test
Changes