-
Notifications
You must be signed in to change notification settings - Fork 13.6k
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
core: Move document loader interfaces to core #17723
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
2e45ee1
to
3513ec1
Compare
There's a circular dependency since BaseLoader uses TextSplitter which is in the langchain package. How should it be handled ? Maybe move TextSplitter and RecursiveCharacterTextSplitter to core ? |
80a930e
to
4f3a297
Compare
I opted for moving TextSplitter and RecursiveCharacterTextSplitter to core. |
7b6959d
to
deb6241
Compare
#18346 moved the text splitters to their own package. |
deb6241
to
8c59c9f
Compare
8c59c9f
to
79e9224
Compare
79e9224
to
588ff5e
Compare
Rebased to use langchain-text-splitters |
LGTM, thank you for identifying the problem and coming up with an elegant solution! |
Fantastic I've been wanting to do this for a while! |
@baskaryan this looks good to me. We need your input on the namespaces. Do we match |
OK document loaders is good according to @baskaryan |
comandeering to resolve conflicts |
This is needed to be able to move document loaders to partner packages. --------- Co-authored-by: Eugene Yurtsev <eyurtsev@gmail.com>
This is needed to be able to move document loaders to partner packages. --------- Co-authored-by: Eugene Yurtsev <eyurtsev@gmail.com>
This is needed to be able to move document loaders to partner packages.