Skip to content

Conversation

@emeryberger
Copy link
Member

No description provided.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR updates support for new Anthropic Claude model variants by adding rate limit entries, adjusting default model selection, and updating token handling.

  • Reformats and extends MODEL_RATE_LIMITS with new Anthropic Claude models.
  • Modifies token rate limiting and counting logic to recognize Anthropic providers.
  • Changes the default model under AWS to the new Claude-3-5-sonnet-v2 variant.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
src/coverup/llm.py Reformatted rate limits dict, added Anthropic model entries, updated token limit/count logic.
src/coverup/coverup.py Updated default_model() to return the new anthropic.claude-3-5-sonnet-20241022-v2:0 for AWS.
Comments suppressed due to low confidence (1)

src/coverup/llm.py:82

  • The count_tokens function lacks a docstring; add one to explain its purpose, parameters, and return value, especially describing how different providers are handled.
def count_tokens(model_name: str, completion: dict):

Copy link
Collaborator

@jaltmayerpizzorno jaltmayerpizzorno left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would be better not to hardcode that string...

@emeryberger emeryberger merged commit f10d00a into main Jun 10, 2025
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants