Skip to content

Conversation

@hwchase17
Copy link
Contributor

add analyze document chain, which does text splitting and then analysis

Copy link
Contributor

@scadEfUr scadEfUr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

code lgtm barring merge conflict!

just to make sure i understand -- when would you recommend using the AnalyzeDocumentChain with a qa_chain, instead of just using a qa_chain directly?

@hwchase17 hwchase17 merged commit 87fad8f into master Feb 7, 2023
@hwchase17 hwchase17 deleted the harrison/document-split branch February 7, 2023 04:02
@blob42 blob42 mentioned this pull request Feb 21, 2023
zachschillaci27 pushed a commit to zachschillaci27/langchain that referenced this pull request Mar 8, 2023
add analyze document chain, which does text splitting and then analysis
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants