Skip to content

feat:support AWS bedrock calling claude#76

Merged
meguminnnnnnnnn merged 12 commits intocloudwego:mainfrom
FamousMai:main
Feb 28, 2025
Merged

feat:support AWS bedrock calling claude#76
meguminnnnnnnnn merged 12 commits intocloudwego:mainfrom
FamousMai:main

Conversation

@FamousMai
Copy link
Copy Markdown
Contributor

@FamousMai FamousMai commented Jan 22, 2025

What type of PR is this?

Check the PR title.

  • This PR title match the format: <type>(optional scope): <description>
  • The description of this PR title is user-oriented and clear enough for others to understand.
  • Attach the PR updating the user documentation if the current PR requires user awareness at the usage level. User docs repo

(Optional) Translate the PR title into Chinese.

新增支持Aws bedrock方式调用cluade

(Optional) More detailed description for this PR(en: English/zh: Chinese).

en:
zh(optional):

(Optional) Which issue(s) this PR fixes:

(optional) The PR that updates user documentation:

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Jan 22, 2025

CLA assistant check
All committers have signed the CLA.

Comment thread libs/acl/claude/chat_model.go Outdated
Comment thread components/model/claude/claude.go
Comment thread components/model/claude/claude.go
Comment thread components/model/claude/claude.go Outdated
Comment thread components/model/claude/claude.go Outdated
Comment thread components/model/claude/claude.go Outdated
Comment thread components/model/claude/claude.go
@FamousMai
Copy link
Copy Markdown
Contributor Author

#76 (comment)

这个文档信息是准确的吗?

是准确的,实际上要申请到这个key流程上会有些许麻烦,所以我感觉只能放一个引导去开通的文档链接

@meguminnnnnnnnn
Copy link
Copy Markdown
Contributor

这个解下冲突吧

@meguminnnnnnnnn
Copy link
Copy Markdown
Contributor

Uploading image.png…
这个签署下

@FamousMai
Copy link
Copy Markdown
Contributor Author

@meguminnnnnnnnn 你的图片没上传成功吗?看不到,不过我好像之前签署过了

@meguminnnnnnnnn
Copy link
Copy Markdown
Contributor

image
点快了。。。 这个应该后来有更新,需要重新签下

@FamousMai
Copy link
Copy Markdown
Contributor Author

image 点快了。。。 这个应该后来有更新,需要重新签下

ok了,签署了

Comment thread components/model/claude/claude.go
Comment thread components/model/claude/claude.go Outdated
Comment thread components/model/claude/claude.go Outdated
Comment thread components/model/claude/claude.go Outdated
@FamousMai
Copy link
Copy Markdown
Contributor Author

不知道为啥 我的账号回复不了你上面的review评论
image

这个你能看到吗
@meguminnnnnnnnn

@meguminnnnnnnnn
Copy link
Copy Markdown
Contributor

不知道为啥 我的账号回复不了你上面的review评论 image

这个你能看到吗 @meguminnnnnnnnn

看不到,需要提交了我才能看到

@meguminnnnnnnnn
Copy link
Copy Markdown
Contributor

image
点这里

FamousMai

This comment was marked as duplicate.

Comment thread components/model/claude/claude.go Outdated
@meguminnnnnnnnn
Copy link
Copy Markdown
Contributor

image
这个pr是你和另一个人合作的嘛,他也需要签署下

@FamousMai
Copy link
Copy Markdown
Contributor Author

image 这个pr是你和另一个人合作的嘛,他也需要签署下

这个邮箱也是我,我没注意,我把重新变基把这个邮箱去掉

@FamousMai
Copy link
Copy Markdown
Contributor Author

@meguminnnnnnnnn 大佬,你说的我都改了,麻烦你了,那么有耐心[捂脸]

@meguminnnnnnnnn
Copy link
Copy Markdown
Contributor

@meguminnnnnnnnn 大佬,你说的我都改了,麻烦你了,那么有耐心[捂脸]

没事,你来做贡献,该我谢你😂,

@meguminnnnnnnnn meguminnnnnnnnn merged commit 18de6fb into cloudwego:main Feb 28, 2025
@FamousMai
Copy link
Copy Markdown
Contributor Author

@meguminnnnnnnnn 大佬,你说的我都改了,麻烦你了,那么有耐心[捂脸]

没事,你来做贡献,该我谢你😂,

跟你们多学习!后面我们也会用 eino 来搞事情,我还会再贡献的!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants