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

Streaming chat in DevUI #442

Closed
geoand opened this issue Apr 3, 2024 · 8 comments · Fixed by #590
Closed

Streaming chat in DevUI #442

geoand opened this issue Apr 3, 2024 · 8 comments · Fixed by #590
Labels
enhancement New feature or request

Comments

@geoand
Copy link
Collaborator

geoand commented Apr 3, 2024

Would it be possible to use a StreamingChatModel in the DevUI if it's accessible?
This would allow to stream the response back

@geoand geoand added the enhancement New feature or request label Apr 3, 2024
@geoand
Copy link
Collaborator Author

geoand commented Apr 4, 2024

@jmartisk do you think this is possible without too much work?

@jmartisk
Copy link
Collaborator

jmartisk commented Apr 4, 2024

What would that mean exactly, that the response from the model is a Multi and we would forward the events from the Multi to the UI as they arrive, over a websocket?

@geoand
Copy link
Collaborator Author

geoand commented Apr 4, 2024

Yes

@jmartisk
Copy link
Collaborator

jmartisk commented Apr 4, 2024

Yeah that shouldn't be very hard. I can look into it, just not sure when, things are piling up now :)

@geoand
Copy link
Collaborator Author

geoand commented Apr 4, 2024

Yeah, not a priority for sure, it would just be nice for demos

@iocanel
Copy link
Collaborator

iocanel commented May 15, 2024

I can give it a shot.

@iocanel
Copy link
Collaborator

iocanel commented May 15, 2024

I have something is working. However, it will only work when rag is disabled (and it's true by default).

Should we add a message / alert to let the user know when streaming chat is not enabled and why it's not enabled?
Would you think that a vaadin alert or a console.log is more appropriate for this?

@geoand
Copy link
Collaborator Author

geoand commented May 15, 2024

Should we add a message / alert to let the user know when streaming chat is not enabled and why it's not enabled?

I think that's a good idea.

I am pretty sure that noone is going to look at the browser console though :)

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

Successfully merging a pull request may close this issue.

3 participants