Skip to content

Modernize 2-retrieval-qa-finish for LangChain v1 compatibility#8

Open
Flcookie wants to merge 1 commit into
emarco177:2-retrieval-qa-finishfrom
Flcookie:update-2-retrieval-qa-modern
Open

Modernize 2-retrieval-qa-finish for LangChain v1 compatibility#8
Flcookie wants to merge 1 commit into
emarco177:2-retrieval-qa-finishfrom
Flcookie:update-2-retrieval-qa-modern

Conversation

@Flcookie
Copy link
Copy Markdown

Summary

This PR updates the 2-retrieval-qa-finish branch to be compatible with the latest LangChain v1 ecosystem.

Changes

  • ingestion.py

    • Updated imports to langchain_text_splitters, langchain_openai, and langchain_pinecone.
    • Added safe metadata handling and clearer argument naming.
  • backend/core.py

    • Updated imports to use langchain_classic for create_retrieval_chain and create_stuff_documents_chain.
    • Preserved all educational structure and function names for the course.

Motivation

Keep the course code runnable under LangChain v1 while maintaining its original teaching structure and clarity.

Notes

If no local folder langchain-docs/... exists, ingestion.py will load 0 documents — this is expected.


🙏 Acknowledgment
A huge thanks to the course author and contributors for creating such a clear and practical learning project.
It has been incredibly helpful for understanding how to build retrieval-augmented generation pipelines with LangChain.

@Flcookie
Copy link
Copy Markdown
Author

Thanks for this amazing course and project! It really helped me understand RAG and LangChain in depth 🙏

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.

1 participant