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

[BUG] Chat history should be manage at backend if ChatId or Session id is getting passed in api calls. #1666

Closed
navinkeshari opened this issue Feb 2, 2024 · 6 comments
Labels
enhancement New feature or request in-work Issue in work

Comments

@navinkeshari
Copy link

Construct a sequence involving memory nodes and conversation nodes. Initiate a conversation via the REST API, providing the chat ID or session ID after the initial call response. The flow fails to maintain the same context and instead alters the context. To handle context changes, the backend requires the passing of a chat_history list, potentially leading to an increase in request bandwidth. The system should retrieve chat details based on the chat or session ID.

@HenryHengZJ
Copy link
Contributor

agree! we will work on that improvement!

@emrahtoy
Copy link
Contributor

emrahtoy commented Feb 5, 2024

also related with #1584

@HenryHengZJ HenryHengZJ added the enhancement New feature or request label Feb 6, 2024
@yos1p
Copy link

yos1p commented Mar 4, 2024

Thumbs up on this. Very important.

@HenryHengZJ
Copy link
Contributor

This is now available in v1.6.5, every memory nodes now have a sessionId and automatically retrieve the chat history with it. This removes the need for passing history array in the API request

@emrahtoy
Copy link
Contributor

Much appreciated, thank you.

@haimco50
Copy link

This is now available in v1.6.5, every memory nodes now have a sessionId and automatically retrieve the chat history with it. This removes the need for passing history array in the API request

thanks! @HenryHengZJ - what about context field on any other node fields initialized as part of any sessionId conversation - are those will be resored as well when passing a sessionId in a new conversation - trying to figure out how session tracking realy works - it seems to be working for chat hostory but not for context field for example - not sure whether this is bug or by design.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request in-work Issue in work
Projects
None yet
Development

No branches or pull requests

5 participants