-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
refactor: implementing the historical context of LangChainBot using the BufferMemory encapsulated by LangChain #537
Conversation
… the BufferMemory encapsulated by LangChain.
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Thank you very much! Introducing more LangChain features are exactly in the plan. I have only one consideration. The Chat Title is not always unique. Is it a good choice of key? How about continuing using chatContext to restore messages? |
… the BufferMemory encapsulated by LangChain.
Yes, thank you for your suggestion. I have revised the code again, reducing the amount of previous modifications. Perhaps my previous modification ideas were too greedy, hahaha |
… the BufferMemory encapsulated by LangChain.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for more commits. I have two comments on the latest code.
… the BufferMemory encapsulated by LangChain.
Thanks! You nailed it with those suggestions. I've fixed them up. |
Thank you very much. The PR will be included in next release. |
Optimize by implementing the historical context of LangChainBot using the BufferMemory encapsulated by LangChain.