-
-
Notifications
You must be signed in to change notification settings - Fork 232
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Added Llama Bedrock client #1943
Conversation
Please retry analysis of this Pull-Request directly on SonarCloud |
Please retry analysis of this Pull-Request directly on SonarCloud |
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.
It's working well!
By the way I found a small error in Claude Client, line 91.
we should replace
sampled_tokens=completion["usage"]["input_tokens"]
with
sampled_tokens=completion["usage"]["output_tokens"]
Also, it would be better to have a common class called |
Please retry analysis of this Pull-Request directly on SonarCloud |
Please retry analysis of this Pull-Request directly on SonarCloud |
Description
Added Llama Bedrock client
Related Issue
Type of Change
Checklist
CODE_OF_CONDUCT.md
document.CONTRIBUTING.md
guide.pdm.lock
runningpdm update-lock
(only applicable whenpyproject.toml
has beenmodified)