-
Notifications
You must be signed in to change notification settings - Fork 47
Add Session Management Feat. #170
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
Comments
It's under development and should be available soon. |
@ajac-zero work is available on this branch/PR |
Awesome! The semantic session manager is really interesting. I noticed each exchange is expected to be a pair of strings, would this limit the use of tool messages? Due to most providers returning tool/function calls as dicts. I suppose the simplest solution would be to stringify the response dicts before storing them, although this might require additional parsing. Alternatively using json mode would work fine already if available. Also, there are situations where a single exchange could involve more than 2 messages. Like when a llm uses tools before generating a response. user query message -> assistant tool call message -> tool result message -> assistant response message For cases like these it could be useful to provide the option to store messages individually to allow greater flexibility. |
Hi @ajac-zero. The session managers do have a convenience |
Has progress begun on llm session management? It would be an awesome feature.
Currently, our apps pull past conversation messages straight from our db upon each request, identified by a conversation id.
However, saving on-going conversations in-cache would be much faster and off-load some work from the db.
Is there an idea on how this would be implemented? Or should I pr something from scratch?
The text was updated successfully, but these errors were encountered: