-
Notifications
You must be signed in to change notification settings - Fork 45
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
Feature/upgrade langchain dependency from 0.0332 to 0.2.5 #394
Feature/upgrade langchain dependency from 0.0332 to 0.2.5 #394
Conversation
d5d4df5
to
7b2adbd
Compare
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 for getting the upgrade done! I think can really help us move forward with LangChain versions. Please check some minor questions above and I think we are good to merge this one.
96aee0c
to
d3016fe
Compare
d3016fe
to
c0c476c
Compare
@saminegash Please see some responses above. |
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.
LGTM now
Description
Type of change
Related issues
Checklists
To speed up the review process, please follow these checklists:
Development
make format && make lint
)make test
)See the testing guidelines for help on tests, especially those involving web services.
Code review
💔 Thank you for submitting a pull request!