-
Notifications
You must be signed in to change notification settings - Fork 4k
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
[Bug]: The Anthropic provider does not support vision #3792
Comments
I'll test for openAI in a bit. FWIW I am not sure FE should change this, it feels like the right thing to try as best as we can to standardize with provider. More like, this pattern could be added to liteLLM's list, since it supports it, it just doesn't know it has vision if we write it this way, but it does know if we write it without "anthropic/"... hah. Maybe we can fix it somehow first, though, pending theirs. |
Looking at the list of prices, this would be broken for OpenAI as well: All OAI models do not have a provider/ prefix. I think it would make sense to request the additions within the litellm repo to standardize and prevent the need to maintaining a mapping within this repo. Created an issue here: BerriAI/litellm#5608 |
I've made a pull request in litellm that will resolve this: |
@ColeMurray That is awesome! Thanks a lot |
That's really super, hopefully it'll get the ✅ soon. Thanks @ColeMurray ! |
Is there an existing issue for the same bug?
Describe the bug
The current model ID's used are incorrect and do not support vision.
Current OpenHands version
Installation and Configuration
Model and Agent
Anthropic models
Operating System
osx
Reproduction Steps
Logs, Errors, Screenshots, and Additional Context
No response
The text was updated successfully, but these errors were encountered: