-
Notifications
You must be signed in to change notification settings - Fork 7.1k
Add o1 to verfied models #6642
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
Add o1 to verfied models #6642
Conversation
Does it need to be added anywhere in this file? OpenHands/openhands/llm/llm.py Lines 44 to 98 in 6655ec0
|
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.
I don't think it needs to be in more places in llm.py.
Maybe we can remove o1-preview
from verified, if we add o1
?
You know better. Should I remove it? |
Yes, I don't think there's a great use anymore, and anyway people can add it themselves in advanced settings |
End-user friendly description of the problem this fixes or functionality that this introduces
Add o1 to verified models
Give a summary of what the PR does, explaining any non-trivial design decisions
Tested setting provider to OpenAI and model to o1 and it works. Do we want it in the "verified models"?
Link of any specific issues this addresses
#6030
To run this PR locally, use the following command: