Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat[community]: Add chat deployment to IBM chat class #7633

Merged
merged 3 commits into from
Feb 5, 2025

Conversation

FilipZmijewski
Copy link
Contributor

This PR adds chat deployment options to Watsonx IBM class allowing user to use langchain with their deployed models

* Rename auth method in docs

* Rename auth method in docs

* Add deployment chat to chat class

* Upadate Watsonx sdk

* Rework interfaces in llms as well

* Bump watsonx-ai sdk version

* Remove unused code

* Add fake auth
Copy link

vercel bot commented Jan 30, 2025

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

Name Status Preview Comments Updated (UTC)
langchainjs-docs ✅ Ready (Inspect) Visit Preview Feb 4, 2025 1:12pm
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
langchainjs-api-refs ⬜️ Ignored (Inspect) Feb 4, 2025 1:12pm

@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Jan 30, 2025
@FilipZmijewski FilipZmijewski changed the title feature[community]: Add chat deployment to IBM chat class feat[community]: Add chat deployment to IBM chat class Jan 30, 2025
@dosubot dosubot bot added the auto:enhancement A large net-new component, integration, or chain. Use sparingly. The largest features label Jan 30, 2025
Copy link
Collaborator

@jacoblee93 jacoblee93 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Happy to merge as is but see comments

@dosubot dosubot bot added size:XL This PR changes 500-999 lines, ignoring generated files. and removed size:L This PR changes 100-499 lines, ignoring generated files. labels Feb 4, 2025
@FilipZmijewski
Copy link
Contributor Author

Thanks for your suggestions. Fixed it so ready to merge!

Copy link
Collaborator

@jacoblee93 jacoblee93 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@dosubot dosubot bot added the lgtm PRs that are ready to be merged as-is label Feb 5, 2025
@jacoblee93 jacoblee93 merged commit 99829ef into langchain-ai:main Feb 5, 2025
31 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
auto:enhancement A large net-new component, integration, or chain. Use sparingly. The largest features lgtm PRs that are ready to be merged as-is size:XL This PR changes 500-999 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants