-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Add last_speaker to GroupChatManager #3318
Conversation
Thanks @thinkall, I've tested these changes and they look good. It may me good to add it somewhere in the documentation because I don't think it will show, even in the API reference, otherwise. The other option is to create it as a function on GroupChatManager, e.g. |
Thank you very much for the suggestions, @marklysze ! I've added the docstring. Instead of |
* Add last_speaker to GroupChatManager's property * Add docstring for last_speaker * Format docstring
Why are these changes needed?
Add a property
last_speaker
forGroupChatManger
, so users can have easier access to the info of speakers. The use case is discussed in #3279Related issue number
Closes #3279
Checks