-
Notifications
You must be signed in to change notification settings - Fork 38
INTPYTHON-828 - Make Langchain-MongoDB compatible with LangChain >= 1.0 #271
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
Conversation
|
|
||
| ## Changes in version 0.8.0 (2025/11/18) | ||
|
|
||
| - Update dependencies to align with LangChain 1.0, which is now the minimum supported version of LangChain. |
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 think we should call out that we depend on langchain_classic now, since it is a heavy dependency that might not be obvious why it ended up in a user's environment.
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 think we should call out that we depend on
langchain_classicnow, since it is a heavy dependency that might not be obvious why it ended up in a user's environment.
These same dependencies (or close to) already exist in langchain < 1.0. What I would mention is that. "Note: langchain_mongodb 0.8.0 depends on langchain_classic, so still has the heavy dependency set than langchain did before the v1 refactor."
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.
Is the updated wording sufficient?
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.
Did you call out langchain-classic?
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.
The latest commit does.
blink1073
left a comment
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!
|
|
||
| ## Changes in version 0.8.0 (2025/11/18) | ||
|
|
||
| - Update dependencies to align with LangChain 1.0, which is now the minimum supported version of LangChain. |
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.
Did you call out langchain-classic?
|
Looking into the test failure for |
caseyclements
left a comment
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
INTPYTHON-828
Patch build (updated): https://spruce.mongodb.com/version/691cbf67fd4ec10007a99531/tasks?sorts=STATUS%3AASC%3BBASE_STATUS%3ADESC