-
Notifications
You must be signed in to change notification settings - Fork 141
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
Bedrock対応 #75
Bedrock対応 #75
Conversation
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.
ありがとうございます!現状 LGTM です!
GA 後すぐにマージという意味では、こちらの PR でドキュメントも更新しちゃっても良いと思います!:pray:
packages/cdk/lambda/bedrockApi.ts
Outdated
import aws4Interceptor from 'aws4-axios'; | ||
import axios from 'axios'; | ||
import { UnrecordedMessage } from 'generative-ai-use-cases-jp'; | ||
import { IncomingMessage } from 'http'; |
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.
現状 OK ですが、GA されたら region は configurable or Lambda のリージョンとしないとですね。(Tokyo GA されたのに us-east-1 使っていたら悲しいので...w)
packages/cdk/lambda/predictStream.ts
Outdated
const event = new EventStreamCodec(toUtf8, fromUtf8).decode(chunk); | ||
if ( | ||
event.headers[':event-type'].value !== 'chunk' || | ||
event.headers[':content-type'].value !== 'application/json' |
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.
へー bedrock はいちいち json でくるんですねー
@@ -12,8 +12,7 @@ Generative AI(生成系 AI)は、ビジネスの変革に革新的な可能 | |||
|--------------------------------------------------------------------------------------------------------------|------------------------------------------------------------------------------------------------------| | |||
| [![デプロイ手順](https://img.youtube.com/vi/9sMA17OKP1k/0.jpg)](https://www.youtube.com/watch?v=9sMA17OKP1k) | [![デモ](https://img.youtube.com/vi/rkKZZSuVZUU/0.jpg)](https://www.youtube.com/watch?v=rkKZZSuVZUU) | | |||
|
|||
> - 2023年8月現在、Amazon Bedrock はプレビュー版であるため、このリポジトリの実装には含まれていません。OpenAI を LLM としてサポートしていますので、そちらをご利用いただけます。Amazon Bedrock が GA になり次第、対応いたします。 | |||
> - OpenAI を LLM として利用する場合は、別途利用料金がかかります。OpenAI 利用に関する情報は、事前に [OpenAI](https://platform.openai.com/) で確認してください。 | |||
> **2023/09/29 現在、東京リージョンで Amazon Bedrock を利用することができませんので、バージニア北部 (us-east-1) リージョンを利用する設定にしています。こちらの [Model access 画面](https://us-east-1.console.aws.amazon.com/bedrock/home?region=us-east-1#/modelaccess) を開き、「Edit」 → 「Anthropic Claude にチェック」 → 「Save changes」 と操作していただいて、バージニア北部リージョンにて Amazon Bedrock (基盤モデル: Claude) を利用できる状態にしてください** |
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.
👍
@@ -23,8 +22,7 @@ Generative AI(生成系 AI)は、ビジネスの変革に革新的な可能 | |||
|
|||
> :white_check_mark: ... 実装されている、:construction: ... まだ実装されていない | |||
|
|||
- :white_check_mark: OpenAI を LLM として利用 | |||
- :construction: Amazon Bedrock を LLM として利用 | |||
- :white_check_mark: Amazon Bedrock を LLM として利用 |
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.
👍
LGTM!! |
Issue #, if available:
#2
Description of changes:
Bedrock が GA されてから、main にマージする予定。
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.