-
Notifications
You must be signed in to change notification settings - Fork 63
feat: aws bedrock llm implementation #50
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
Conversation
Please add a short video showcasing the passing testcases |
@mschuwalow Hi here the link to the video https://drive.google.com/file/d/1LHxIgjyz8l7pM0uAUq3J_8D3caLoUJG2/view?usp=sharing showcasing the testing and I was facing some problem related golem. Which I talked about on hackathon channel https://discord.com/channels/1134448700572319785/1157293164512690267/1389291819925246093 |
All test passes individually but when run in sync one after one then test 3rd or some times 4th fails. I guess it is may be because my system configurations |
@mschuwalow @vigoo Please check here
|
@mschuwalow Please check this demo video bedrock.2.mp4 |
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.
This is a very reasonable implementation on top of the raw HTTP client.
I'll discuss with my colleagues whether we prefer this approach or the one in #27 from a maintenance perspective.
Closing as we chose to merge the other attempt (#27). Thank you for your work! |
Added AWS bedrock via its converse api link
closes #2
/claim #2