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

Sagemaker endpoint capability to inject boto3 client for cross account scenarios #10728

Merged
merged 1 commit into from
Sep 19, 2023
Merged

Conversation

vikramshitole
Copy link
Contributor

@vercel
Copy link

vercel bot commented Sep 18, 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) Visit Preview Sep 18, 2023 7:15am

@dosubot dosubot bot added Ɑ: models Related to LLMs or chat model modules 🤖:improvement Medium size change to existing code to handle new use-cases 🔌: aws Primarily related to Amazon Web Services (AWS) integrations labels Sep 18, 2023
@baskaryan
Copy link
Collaborator

lgtm, thanks @vikramshitole!

@baskaryan baskaryan added the lgtm PR looks good. Use to confirm that a PR is ready for merging. label Sep 18, 2023
@baskaryan baskaryan merged commit a4e858b into langchain-ai:master Sep 19, 2023
@gretchency
Copy link

I think the sagemaker_endpoint under embedding also need this fix: https://github.com/langchain-ai/langchain/blob/master/libs/langchain/langchain/embeddings/sagemaker_endpoint.py#L50

@vikramshitole
Copy link
Contributor Author

@gretchency raised PR for same #12106

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔌: aws Primarily related to Amazon Web Services (AWS) integrations 🤖:improvement Medium size change to existing code to handle new use-cases lgtm PR looks good. Use to confirm that a PR is ready for merging. Ɑ: models Related to LLMs or chat model modules
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants