-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Support Anthropic as an LLM provider #14614
Conversation
@sdirix Tool functions do not work yet. That is probably in the way how they are parsed from the stream, but I have no clue how that works for openai TBH, maybe you can have a quick look at this. |
3PP Check was approved, should succeed with the next rebuild |
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.
Anthropic works for me, I tested with streaming enabled and disabled.
I also had a look at the tools. The issue is that in our OpenAI language models we directly hand over real Javascript functions into the API which are used by the OpenAI client to resolve the tools, see here.
Anthropic seemingly does not support this but instead expects a new message from the "user", see the first example here.
Suggestion: For the first implementation we skip the tool support and add this in a followup. Of course feel free to add it here too if you want to see it supported.
...tainer/src/electron-node/devcontainer-contributions/main-container-creation-contributions.ts
Outdated
Show resolved
Hide resolved
packages/remote/src/electron-node/setup/remote-setup-service.ts
Outdated
Show resolved
Hide resolved
fixed #14613 Signed-off-by: Jonas Helming <jhelming@eclipsesource.com>
Co-authored-by: Stefan Dirix <sdirix@eclipsesource.com>
Signed-off-by: Jonas Helming <jhelming@eclipsesource.com>
b8a0b8a
to
80dacb3
Compare
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.
Works for me!
fixed #14613
What it does
Adds support for Anthropic as an LLM provider.
How to test
Use Anthropic models.
Breaking changes
Attribution
Review checklist
Reminder for reviewers