Add langchain and transformers to requirements.txt#20
Conversation
Added langchain and transformers dependencies to requirements.
There was a problem hiding this comment.
Pull request overview
This PR updates the backend Python dependency set to include LangChain/Chroma and transformer-based embedding/reranking libraries, aligning with the backend’s existing RAG/knowledge-base services.
Changes:
- Add
transformers,sentence-transformers,langchain,langchain-chroma, andchromadbtobackend/requirements.txt. - Introduce a new dependency block for these AI/RAG-related packages.
Comments suppressed due to low confidence (1)
backend/requirements.txt:51
- There are multiple trailing blank lines added at the end of this file. Please remove the extras to keep diffs clean and avoid churn in future edits.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| #langchain and transformers dependencies | ||
| transformers==4.31.0 |
There was a problem hiding this comment.
The new dependency block is appended after the platform-specific section and the header comment is missing a space after #. To keep this file organized, please move these AI/LLM dependencies up under the existing # AI/LLM section and format the header as # LangChain and transformers dependencies (or similar).
| uvloop==0.22.1; platform_system != "Windows" | ||
|
|
||
| #langchain and transformers dependencies | ||
| transformers==4.31.0 |
There was a problem hiding this comment.
transformers is not imported anywhere in the backend code (only sentence_transformers is). If it’s only a transitive dependency of sentence-transformers, pinning it here increases the chance of resolver conflicts without adding much value. Consider removing the direct transformers==... pin unless you have a concrete runtime requirement for that exact version (in which case, add a short note explaining why).
| transformers==4.31.0 |
Added langchain and transformers dependencies to requirements.