-
Notifications
You must be signed in to change notification settings - Fork 21
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
Anthropic support #48
Conversation
…even with openai set as base provider by overridding base url
This is awesome! I'll take a look on the plane tomorrow. I also added @transitive-bullshit as a reviewer because he likely has valuable insights. |
@cfortuner I was thinking more about this and it seems odd to add Anthropic-specific stuff into this very simple package called |
@rileytomasek yeah that sounds good probably would simplify things if we just brought both of these into dexter rather than trying to support these packages separately, but your call. i'll start a new repo for anthropic-fetch? |
I was thinking, maybe we should just create anthropic fetch, and then handle any of the translation between openai-client + anthropic client in dexter? |
@rileytomasek and I wanted to loop you in on this to get your advice on how we could support anthropic (or other providers) in dexter. What do you think about this?
|
I think this is roughly the correct direction, but can be even simpler. Dexter (and probably most other libs) would prefer to just work with OpenAI types (like What do you think of this:
This way Dexter and |
@rileytomasek 👍 yeah that's exactly what I had in mind |
Adds support for Anthropic for Chat.