Skip to content
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

Fixed validation error when credentials_profile_name, or region_name is not passed #7629

Merged
merged 1 commit into from
Jul 13, 2023

Conversation

3coins
Copy link
Contributor

@3coins 3coins commented Jul 13, 2023

Summary

This PR corrects the checks for credentials_profile_name, and region_name attributes. This was causing validation exceptions when either of these values were missing during creation of the retriever class.

Fixes #7571

Requested reviewers:

@baskaryan

@vercel
Copy link

vercel bot commented Jul 13, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
langchain ⬜️ Ignored (Inspect) Jul 13, 2023 1:23am

@dosubot dosubot bot added the 🤖:bug Related to a bug, vulnerability, unexpected error with an existing feature label Jul 13, 2023
@3coins 3coins marked this pull request as ready for review July 13, 2023 01:58
@baskaryan
Copy link
Collaborator

thanks @3coins!

@baskaryan baskaryan merged commit f11d845 into langchain-ai:master Jul 13, 2023
@cupdike
Copy link

cupdike commented Aug 4, 2023

The code example on this page needs to be updated to reflect the need for region_name:
https://python.langchain.com/docs/integrations/llms/bedrock

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🤖:bug Related to a bug, vulnerability, unexpected error with an existing feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

AmazonKendraRetriever "Could not load credentials" error in latest release
3 participants