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.

Sign up for GitHub

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

Jump to bottom

Refactor rate limiter initialization in LLM provider configuration #2

Merged
merged 1 commit into from
Feb 12, 2025
Merged

Refactor rate limiter initialization in LLM provider configuration #2

merged 1 commit into from
Feb 12, 2025

Conversation

hoangnb24
Copy link
Collaborator

  • Move rate limiter creation to a single location at the beginning of the function
  • Update parameter names for specific providers (e.g., Mistral's model instead of model_name)
  • Fix Google provider's API key parameter to google_api_key
  • Remove redundant configuration options and simplify model initialization

@hoangnb24
Refactor rate limiter initialization in LLM provider configuration
b797f1f 
- Move rate limiter creation to a single location at the beginning of the function
- Update parameter names for specific providers (e.g., Mistral's `model` instead of `model_name`)
- Fix Google provider's API key parameter to `google_api_key`
- Remove redundant configuration options and simplify model initialization
@hoangnb24 hoangnb24 merged commit 175267c into main Feb 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@hoangnb24