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.

Sign up for GitHub

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

Jump to bottom

Streaming #140

Closed
wants to merge 5 commits into from
Closed

Streaming #140

wants to merge 5 commits into from

Conversation

jackitaliano
Copy link
Contributor

Streaming

  • enable streaming via deep-chat
  • with streaming changes, makes it more difficult to parse/save/retrieve files
  • no longer save files when they're inserted into the chat to database, instead leave that to when page is loaded again.
    • may increase load time of page if many images are present, but I think that's an unlikely scenario
  • have to parse individual deltas in responses to check if a sandbox link is passed
    • thankfully, the deltas for links are consistent:
      • starts with: "](", "sandbox"
      • ends with ")"

Resolves #73

@jackitaliano
enable streaming and fix file parsing/replacement
eecc5ba 
- enable streaming via `deep-chat` `connect`
- remove saving assistant files when received. instead leave that to
page load
- parse inserted images from `processImageCallback` by their delta
received from response interceptor
@jackitaliano
Merge branch 'main' into streaming
b82d740
@jackitaliano
remove callOnRunComplete (from attempted solution)
1ff4cda
@jackitaliano
fix an odd case where an image of a sandbox is passed
9af4500 
- because searching for "sandbox" in file link, if an image with
"sandbox" in the name was passed, would kill the rest of the message.
- now check that it is being used within a link first
@jackitaliano
Merge branch 'main' into streaming
a8ecf77
@jackitaliano jackitaliano marked this pull request as draft June 12, 2024 21:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

support streaming messages
1 participant
@jackitaliano