Remove langchain dependency #41

acatav · 2023-09-10T12:55:09Z

Problem

Langhain is unstable dependency, even with the vanilla installation

Take only the parts we need for chunking (with appropriate credit)

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to not work as expected)
This change requires a documentation update
Infrastructure change (CI configs, etc)
Non-code change (docs, etc)
None of the above: (explain here)

Adjust unit tests

igiloh-pinecone

LGTM!

igiloh-pinecone · 2023-09-11T16:06:11Z

context_engine/knoweldge_base/chunker/langchain_text_splitter.py

Since this is copy-pasted, I'm not even going to review this file...

…dency

remove langchain dependency

16234dd

acatav changed the base branch from main to dev September 10, 2023 12:55

acatav requested a review from igiloh-pinecone September 10, 2023 12:56

change credit

786cb88

igiloh-pinecone approved these changes Sep 11, 2023

View reviewed changes

Merge remote-tracking branch 'origin/dev' into remove-langchain-depen…

7fde3df

…dency

igiloh-pinecone enabled auto-merge September 11, 2023 16:12

igiloh-pinecone merged commit 52fd4ee into dev Sep 11, 2023

igiloh-pinecone deleted the remove-langchain-dependency branch September 11, 2023 16:20