-
Notifications
You must be signed in to change notification settings - Fork 16.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
docs, langchain-unstructured: update langchain-unstructured docs and update ustructured-client dependency #25451
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
I was receiving the following error when trying to run
|
That's because your new |
@cbornet, thanks for the comments. CI is still failing with the same error, but I'm unsure what to make of it. Do you have any guidance for me on that? |
I think the CI error is not related to your PR. |
Sounds good. Please let me know if there's anything I need to do to resolve it |
Be more explicit in the docs about having to create an instance of the UnstructuredClient if you want to customize how it is used by the UnstructuredLoader
@cbornet, I've rebased onto master to resolve the merge conflicts that were occurring. Do you think we can get this merged today once the tests pass? |
I don’t have merge permissions on LangChain 🙂. |
Be more explicit in the docs about creating an instance of the UnstructuredClient if you want to customize it versus using sdk parameters with the UnstructuredLoader.
Bump the unstructured-client dependency as discussed here