-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
fix(community): Fix interface of chat deployment IBM and add test for this case #7666
Conversation
* 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
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Skipped Deployment
|
@@ -254,6 +254,7 @@ | |||
"@ibm-cloud/watsonx-ai": "*", | |||
"@lancedb/lancedb": "^0.12.0", | |||
"@langchain/core": ">=0.2.21 <0.4.0", | |||
"@langchain/langgraph": "*", |
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.
Add to dev deps if at all but would prefer to avoid this as it causes issues with build pulling in old core versions
You can add a file to examples/
, which does have a @langchain/langgraph
dep - not a great system but it'll at least catch type errors on repo build
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.
I reworked it with a 'dummy' function, does the same and no import needed. Just pushed the changes Please, have a look
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.
Thanks!
Fixes # (issue)