Skip to content
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

feat: Support Anthropic Calude 3.5 Sonnet on Vertex AI #1000

Merged
merged 1 commit into from
Jun 25, 2024

Conversation

yamitzky
Copy link
Contributor

@yamitzky yamitzky commented Jun 25, 2024

User description

Added support for Claude 3.5 Sonnet on VertexAI with litellm.


PR Type

enhancement


Description

  • Added support for vertex_ai/claude-3-5-sonnet@20240620 with a token limit of 100000 in the pr_agent/algo/__init__.py file.

Changes walkthrough 📝

Relevant files
Enhancement
__init__.py
Add support for Claude 3.5 Sonnet on Vertex AI                     

pr_agent/algo/init.py

  • Added support for vertex_ai/claude-3-5-sonnet@20240620 with a token
    limit of 100000.
  • +1/-0     

    💡 PR-Agent usage:
    Comment /help on the PR to get a list of all available PR-Agent tools and their descriptions

    Copy link
    Contributor

    PR-Agent was enabled for this repository. To continue using it, please link your git user with your CodiumAI identity here.

    PR Reviewer Guide 🔍

    ⏱️ Estimated effort to review [1-5] 1
    🏅 Score 95
    🧪 Relevant tests No
    🔒 Security concerns No
    🔀 Multiple PR themes

    No

    ⚡ Key issues to review None

    Copy link
    Contributor

    PR-Agent was enabled for this repository. To continue using it, please link your git user with your CodiumAI identity here.

    PR Code Suggestions ✨

    CategorySuggestion                                                                                                                                    Score
    Maintainability
    Sort dictionary keys alphabetically to improve readability and maintainability

    Consider sorting the dictionary keys alphabetically to improve readability and
    maintainability, making it easier to locate specific entries.

    pr_agent/algo/init.py [26-32]

    -'vertex_ai/claude-3-haiku@20240307': 100000,
    -'vertex_ai/claude-3-sonnet@20240229': 100000,
    -'vertex_ai/claude-3-opus@20240229': 100000,
    -'vertex_ai/claude-3-5-sonnet@20240620': 100000,
    -'vertex_ai/gemini-1.5-pro': 1048576,
     'codechat-bison': 6144,
     'codechat-bison-32k': 32000,
    +'vertex_ai/claude-3-5-sonnet@20240620': 100000,
    +'vertex_ai/claude-3-haiku@20240307': 100000,
    +'vertex_ai/claude-3-opus@20240229': 100000,
    +'vertex_ai/claude-3-sonnet@20240229': 100000,
    +'vertex_ai/gemini-1.5-pro': 1048576,
     
    • Apply this suggestion
    Suggestion importance[1-10]: 5

    Why: Sorting dictionary keys can indeed improve readability, but Python dictionaries maintain insertion order and are not inherently sorted. The suggestion is valid for maintainability but not crucial for functionality.

    5

    @mrT23
    Copy link
    Collaborator

    mrT23 commented Jun 25, 2024

    lgtm

    @mrT23 mrT23 merged commit 4ab747d into Codium-ai:main Jun 25, 2024
    1 check passed
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Labels
    Projects
    None yet
    Development

    Successfully merging this pull request may close these issues.

    2 participants