Skip to content
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

Add tests and update notebook for MarkdownHeaderTextSplitter #6069

Merged

Conversation

rlancemartin
Copy link
Collaborator

Add test and update notebook for MarkdownHeaderTextSplitter.

@rlancemartin rlancemartin force-pushed the rlm/add_md_header_splitter_test branch from 070fede to 43953a9 Compare June 12, 2023 23:51
Copy link
Contributor

@hwchase17 hwchase17 left a comment

Choose a reason for hiding this comment

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

lets make those separate test cases - easier to isolate failures

@rlancemartin rlancemartin force-pushed the rlm/add_md_header_splitter_test branch from c81dda3 to cbd4c75 Compare June 13, 2023 05:40
@rlancemartin rlancemartin merged commit ee3d051 into langchain-ai:master Jun 13, 2023
Undertone0809 pushed a commit to Undertone0809/langchain that referenced this pull request Jun 19, 2023
…in-ai#6069)

Add test and update notebook for `MarkdownHeaderTextSplitter`.
This was referenced Jun 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants