-
Notifications
You must be signed in to change notification settings - Fork 930
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(llm-proxy): Add support for O1 & other LLMs #1034
Conversation
PR Reviewer Guide 🔍Here are some key observations to aid the review process:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 Looks good to me! Reviewed everything up to 003cc0b in 13 seconds
More details
- Looked at
60
lines of code in1
files - Skipped
0
files when reviewing. - Skipped posting
1
drafted comments based on config settings.
1. llm-proxy/litellm-config.yaml:49
- Draft comment:
The model 'o1' and 'o1-mini' have the same 'litellm_params'. Consider verifying if this is intentional or if 'o1' should have different parameters. - Reason this comment was not posted:
Comment did not seem useful.
Workflow ID: wflow_DV4ZLFMPyia8UDDN
You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet
mode, and more.
PR Code Suggestions ✨Explore these optional code suggestions:
|
PR Type
Enhancement
Description
Added support for new LLM models in
litellm-config.yaml
.Introduced configurations for OpenAI's
o1-mini
model.Added multiple OpenRouter models, including
eva-llama-3.33-70b
andhermes-3-llama-3.1-70b
.Updated the
model_list
with detailed parameters and tags for new models.Changes walkthrough 📝
litellm-config.yaml
Added new LLM models to configuration
llm-proxy/litellm-config.yaml
o1-mini
model.model_list
with tags and API key references.Important
Add support for O1 and several OpenRouter LLMs in
litellm-config.yaml
.o1
model withopenai/o1-mini
parameters.eva-llama-3.33-70b
,l3.3-euryale-70b
,magnum-v4-72b
,eva-qwen-2.5-72b
,hermes-3-llama-3.1-70b
, anddeepseek-chat
models with respective OpenRouter parameters.litellm-config.yaml
to include new models under themodel_list
section with appropriateapi_key
andtags
.This description was created by
for 003cc0b. It will automatically update as commits are pushed.