-
Notifications
You must be signed in to change notification settings - Fork 111
remove ChatManager #387
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
remove ChatManager #387
Conversation
lukasIO
left a comment
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.
makes sense to me, should we have a last 0.x release that deprecates it and offers alternatives?
|
we will release a major version, v1.0 |
|
Whats the alternative here guys? You pulled these api's, the docs are all outdated. Whats the alternative for this or have we reached a point where the community needs to fork? |
Plus one, I wanted to update Livekit deps and now I don't know how to start it Happened when I did poetry.lock diff |
|
I bet it is in many peoples code right now. Deleting it without explanation and message about deprecation is a mistake |
|
I am going to just write the same code myself. The only possible alternative ... |
|
So , what I have done - I just copied the code from the deleted rtc/chat.py to custom_chat_manager.py . And used it like this for agent to answer text messages for example. It worked |
|
Sorry about that. We'll be releasing Agents 1.0 soon, which will offer a number of new features, including the ability to use text streaming for text input. |
|
For anyone coming here like me also wondering why ChatManager was removed - CW graciously explained in slack that ChatManager was "mis-leading" and the wrong layer of abstraction, and it's better to just handle the data frames directly - appreciate the explaination! |
|
apologies for the lack of explanation. we are in the middle of upgrading the whole text stack in LiveKit to TextStreams. the frontend apps will be updated in the next week as well. instead of ChatManager, you could listen to a text stream of if this becomes an issue for you, please feel free to use an older version of |
I think it is a good timing to do it for v1.0