Skip to content
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 support for using Anthropic Claude models through Amazon Bedrock #2038

Open
wants to merge 13 commits into
base: release
Choose a base branch
from

Conversation

Chen188
Copy link

@Chen188 Chen188 commented Apr 8, 2024

Amazon Bedrock is a cloud service provided by AWS, it supports many Fundation Models, through this PR, user can use Anthropic's Claude models through Amazon Bedrock, in both streaming and non-streaming mode.

@@ -36,7 +36,9 @@
"ws": "^8.13.0",
"yaml": "^2.3.4",
"yargs": "^17.7.1",
"yauzl": "^2.10.0"
"yauzl": "^2.10.0",
"@aws-sdk/client-bedrock-runtime": "^3.515.0",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this really necessary? We don't use official backend client packages but do direct HTTP calls instead.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it isn't. bedrock has a rest api

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, Bedrock do have a REST API, but I'd like to recommand using Bedrock SDK, cause:

  • Bedrock has its own complicate authN mechanism(i.e. through SigV4),
  • Customized wrapper over streaming data instead of SSE

There might be much more coding effort to do with REST API, thus offical SDK is a better choice.

@github-actions github-actions bot added the 🚫 Merge Conflicts [PR] Submitted code needs rebasing label Apr 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🚫 Merge Conflicts [PR] Submitted code needs rebasing 🟥 PR - XL
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants