Standardize conversation agents passing in ChatSession #1191
Unanswered
balloob
asked this question in
Entity Models
Replies: 1 comment
-
Generally: Looks good to me. This seems like a good simple step forward as we expand the capabilities of Do we want |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
Context
Conversation agents have a new shared history class called ChatSession. Instead of requiring all conversation agents to request it, we can standardize a new method, in a backwards compatible way to pass this in.
Old method to process an incoming chat from a user:
Proposal
The proposal is to change it to the following:
Conversation agents that leverage ChatSession can migrate their code to use the new
_async_handle_message
method. Unmigrated conversation agents can continue to use theasync_process
method as before.Alternatives
We can inspect the
async_process
method to see if it supports the ChatSession parameter. We can also add a variable to the class to indicate if it implements v1 or v2 of the process API.Beta Was this translation helpful? Give feedback.
All reactions