Skip to content

WIP: Consolidate streaming logic#50

Draft
alexanderatallah wants to merge 7 commits into
mainfrom
alex/consolidate_streaming
Draft

WIP: Consolidate streaming logic#50
alexanderatallah wants to merge 7 commits into
mainfrom
alex/consolidate_streaming

Conversation

@alexanderatallah

Copy link
Copy Markdown
Owner

Unsuccessful attempt.

Bad surprises:

  • a user passing in onStreamResult and numOutputs > 1 will get no stream results
  • some kind of bug happens when doing numOutputs > 1

@vercel

vercel Bot commented Apr 29, 2023

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
window-ai ✅ Ready (Inspect) Visit Preview Apr 29, 2023 5:46pm

@alexanderatallah alexanderatallah marked this pull request as draft April 29, 2023 17:46
@alexanderatallah alexanderatallah changed the title Consolidate streaming logic WIP: Consolidate streaming logic Apr 29, 2023
@alexanderatallah alexanderatallah linked an issue Apr 30, 2023 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

No open projects
Status: In Progress

Development

Successfully merging this pull request may close these issues.

Allow streaming multiple outputs

1 participant