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

Pre langgraph refactor #32

Open
wants to merge 8 commits into
base: fix-stream-problem
Choose a base branch
from

Conversation

helenCNode
Copy link
Collaborator

@helenCNode helenCNode commented Dec 10, 2024

• Simplifies app.py code before deploying Langgraph
• Introduces modified versions of Langchain's create_history_aware_retriever and create_retrieval_chain so that we can pass on arguments to the retriever more easily; there is also a slight modification to CustomRetriever._get_relevant_documents
Note that you now need to add LANGFUSE_USER_ID = "anon" to .env. I wanted to be able to separate out my own testing of Nesta Brain from other people's use

@helenCNode helenCNode requested a review from beingkk December 10, 2024 14:21
@beingkk beingkk changed the base branch from dev to fix-stream-problem December 18, 2024 16:12
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