Skip to content

Add docset.take_stream()#1071

Merged
baitsguy merged 7 commits into
mainfrom
take-streaming
Dec 11, 2024
Merged

Add docset.take_stream()#1071
baitsguy merged 7 commits into
mainfrom
take-streaming

Conversation

@baitsguy
Copy link
Copy Markdown
Contributor

@baitsguy baitsguy commented Dec 10, 2024

Returns the result on execute_iter() instead of collecting responses. Not proud of the unit test but it does function

@baitsguy baitsguy changed the title Add docset.take_streaming() Add docset.take_stream() Dec 10, 2024
@baitsguy baitsguy marked this pull request as ready for review December 10, 2024 03:33
Comment thread lib/sycamore/sycamore/tests/unit/test_docset.py Outdated
Comment thread lib/sycamore/sycamore/tests/unit/test_docset.py Outdated
Comment thread lib/sycamore/sycamore/tests/unit/test_docset.py Outdated
Comment thread lib/sycamore/sycamore/tests/unit/test_docset.py Outdated
Comment thread lib/sycamore/sycamore/tests/unit/test_docset.py Outdated
Comment thread lib/sycamore/sycamore/tests/integration/test_docset.py Outdated
@baitsguy baitsguy enabled auto-merge (squash) December 11, 2024 00:03
@baitsguy baitsguy merged commit f051247 into main Dec 11, 2024
@baitsguy baitsguy deleted the take-streaming branch December 11, 2024 00:19
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.

2 participants