-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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 Bedrock Cloud Model provider #21092
base: main
Are you sure you want to change the base?
Add Bedrock Cloud Model provider #21092
Conversation
…rock errors, while also letting us include it
This still needs to be updated to the new method of managing models, going to spend the next couple of weeks migrating to it. |
Zed maintainers, would appreciate some guidance on how the ZedPro things are setup and if it makes sense to pass the Bedrock models into it. On a cursory reading of the feature, i don't believe it does -- however, i am open to working with the bedrock service team to make it a reality. |
I don't think you need to pass through the bedrock models through the collab service, as Zed would need to run their own bedrock setup. I'd just do the API key / AWS credentials route. |
Unfortunately, our CLA bot is broken, so I have to substitute for it for a while:
|
Will get it done post a legal review of the docs (just want to make sure AWS doesn't have an issue with it) |
Closes #16544
Release Notes: