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

idea: [DownloadEvent] Adding websocket to publish download event to client #1393

Closed
Tracked by #1242
namchuai opened this issue Oct 3, 2024 · 2 comments · Fixed by #1365
Closed
Tracked by #1242

idea: [DownloadEvent] Adding websocket to publish download event to client #1393

namchuai opened this issue Oct 3, 2024 · 2 comments · Fixed by #1365
Assignees
Milestone

Comments

@namchuai
Copy link
Contributor

namchuai commented Oct 3, 2024

Problem Statement

  • Currently we don't send any event to client. So, client can't update UI accordingly.
  • Drogon does not support SSE.

Feature Idea

  • Publishing events via websocket at ws://127.0.0.1:3928/events
@namchuai namchuai self-assigned this Oct 3, 2024
@namchuai namchuai added this to Menlo Oct 3, 2024
@github-project-automation github-project-automation bot moved this to Investigating in Menlo Oct 3, 2024
@namchuai namchuai moved this from Investigating to In Progress in Menlo Oct 3, 2024
@namchuai namchuai moved this from In Review to Review + QA in Menlo Oct 16, 2024
@namchuai
Copy link
Contributor Author

PR merged, closed!

@gabrielle-ong
Copy link
Contributor

QAing as overall of #1242, marking as complete

  • web sockets events emitted when model pull starts /events
  • web sockets events stopped when model pull stops /events
image

@gabrielle-ong gabrielle-ong moved this from Review + QA to Completed in Menlo Oct 18, 2024
@gabrielle-ong gabrielle-ong added this to the v1.0.1 milestone Oct 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants