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

Add a way to stream output to a UI via sockets #1367

Closed
wants to merge 3 commits into from

Conversation

tyler-suard-parker
Copy link
Contributor

@tyler-suard-parker tyler-suard-parker commented Jan 22, 2024

@sonichi @davorrunje

Why are these changes needed?

Because it is nice to use AutoGen as a backend and stream to a frontend

I added a Singleton, which can be used by multiple scripts. The singleton gets defined in the main script, and we store a socket connection in one of its variables. Then we can easily pass that socket connection down to oai\client.py and use that script to stream back text to a UI.

Related issue number

Closes Issue #1199

Checks

@ekzhu
Copy link
Collaborator

ekzhu commented Jan 28, 2024

@tyler-suard-parker thanks for the PR, there is another PR that seems to address the same issue. I wrote some comment there. Do you want to take a look? #1414 and perhaps join forces?

@tyler-suard-parker
Copy link
Contributor Author

PR closed: using a singleton would prevent multiple users from accessing the same Autogen instance running on a server.
@ekzhu Thank you, I am also @Tylersuard, so I will be happy to join forces with myself 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants