langchain icon indicating copy to clipboard operation
langchain copied to clipboard

text-splitter[minor]: Declare MarkdownHeaderTextSplitter deprecated. Use MarkdownTextSplitter.

Open pprados opened this issue 8 months ago • 7 comments

Langchain offers two Markdown splitter implementations:

  • MarkdownTextSplitter
  • MarkdownHeaderTextSplitter

The MarkdownHeaderTextSplitter class is not langchain compliant, since it inherits from anyone. It is not possible to use in a function or class that expects a TextSplitter.

In my experience, users who come across this class don't know that there is another exists. If they need to be TextSplitter compatible, they write a new class.

We propose to declare it deprecated, and referencing MarkdownTextSplitter.

pprados avatar Jun 10 '24 07:06 pprados