-
Notifications
You must be signed in to change notification settings - Fork 2k
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
feat: AsyncPipeline that can schedule components to run concurrently #8812
Merged
Merged
Changes from all commits
Commits
Show all changes
99 commits
Select commit
Hold shift + click to select a range
9c34772
add component checks
mathislucka fc2f2da
pipeline should run deterministically
mathislucka c28353d
add FIFOQueue
mathislucka 64f4afc
add agent tests
mathislucka 966552e
add order dependent tests
mathislucka 3d0e948
run new tests
mathislucka 7664254
remove code that is not needed
mathislucka 932718f
test: intermediate from cycle outputs are available outside cycle
mathislucka acce8cd
add tests for component checks (Claude)
mathislucka 21c78b8
adapt tests for component checks (o1 review)
mathislucka cf23b32
chore: format
mathislucka 54d4f2c
remove tests that aren't needed anymore
mathislucka 05ed852
add _calculate_priority tests
mathislucka 93f7a9d
revert accidental change in pyproject.toml
mathislucka bbba1b2
test format conversion
mathislucka c00f8c5
adapt to naming convention
mathislucka 235aa47
chore: proper docstrings and type hints for PQ
mathislucka 99ea5d5
format
mathislucka 7577d9b
add more unit tests
mathislucka 25c64cd
rm unneeded comments
mathislucka 5a1bbd8
test input consumption
mathislucka 6820fc3
lint
mathislucka 6a9cdcc
Merge branch 'main' into fix/pipeline_run
mathislucka d624e57
fix: docstrings
mathislucka e1912f8
lint
mathislucka acc17b1
format
mathislucka 207eaba
format
mathislucka 15611fc
fix license header
mathislucka 87010dd
fix license header
mathislucka 7027572
add component run tests
mathislucka 02c82b8
fix: pass correct input format to tracing
mathislucka 7e81ff9
fix types
mathislucka b5db015
Merge branch 'main' into fix/pipeline_run
mathislucka d2bee24
format
mathislucka ad16403
format
mathislucka 2984fcb
types
mathislucka 64a0125
add defaults from Socket instead of signature
mathislucka b2b8adc
fix test names
mathislucka 6b25825
still wait for optional inputs on greedy variadic sockets
mathislucka 2566cb8
fix format
mathislucka ef8e754
wip: warn for ambiguous running order
mathislucka 0252450
wip: alternative warning
mathislucka 580e4bd
fix license header
mathislucka fd32f77
Merge branch 'main' of https://github.com/deepset-ai/haystack into fi…
Amnah199 9426bbb
make code more readable
mathislucka ad1b0aa
Introduce content tracing to a behavioral test
Amnah199 554a6ef
Merge branch 'fix/pipeline_run' of https://github.com/deepset-ai/hays…
Amnah199 a38bccb
Fixing linting
Amnah199 0ac82b2
Remove debug print statements
Amnah199 339d340
Fix tracer tests
Amnah199 9441320
Merge branch 'refs/heads/main' into fix/pipeline_run
mathislucka ae295f7
remove print
mathislucka 592492e
test: test for component inputs
mathislucka cfdc3dc
test: remove testing for run order
mathislucka 25a4b40
chore: update component checks from experimental
mathislucka 74657cd
chore: update pipeline and base from experimental
mathislucka 3b8d886
refactor: remove unused method
mathislucka 00ed49d
refactor: remove unused method
mathislucka f1d325a
refactor: outdated comment
mathislucka 711c846
refactor: inputs state is updated as side effect
mathislucka fa2b65c
format
mathislucka 8b7d761
test: add file conversion test
mathislucka dc6acb9
format
mathislucka ba81a24
fix: original implementation deepcopies outputs
mathislucka c905881
lint
mathislucka 24e41e9
Merge branch 'main' into fix/pipeline_run
mathislucka e045e90
fix: from_dict was updated
mathislucka ed7c700
fix: format
mathislucka 558050f
fix: test
mathislucka 09965ba
test: add test for thread safety
mathislucka a3eb8da
remove unused imports
mathislucka 4d9845a
format
mathislucka 2d11923
test: FIFOPriorityQueue
mathislucka d6673fe
chore: add release note
mathislucka a48d802
feat: add AsyncPipeline
mathislucka fe1e1f1
chore: Add release notes
mathislucka 99e8da3
fix: format
mathislucka 267bbb9
debug: switch run order to debug ubuntu and windows tests
mathislucka 2695163
fix: consider priorities of other components while waiting for DEFER
mathislucka 1ab79cb
refactor: simplify code
mathislucka 0f393ae
Merge branch 'main' into fix/pipeline_run
mathislucka c3f4c72
fix: resolve merge conflict with mermaid changes
mathislucka e0d2712
fix: format
mathislucka 3ded49f
fix: remove unused import
mathislucka 4564c3a
refactor: rename to avoid accidental conflicts
mathislucka 401d5c0
Merge branch 'fix/pipeline_run' into feat/async_pipeline
mathislucka 7a5be08
Merge branch 'main' into feat/async_pipeline
mathislucka e9f4353
fix: track pipeline type
mathislucka e9fbf7b
fix: and extend test
mathislucka 56d2853
fix: format
mathislucka f2bde4f
style: sort alphabetically
mathislucka ff57a5a
Merge branch 'main' into feat/async_pipeline
davidsbatista 06cfad9
Update test/core/pipeline/features/conftest.py
mathislucka 54f33eb
Update test/core/pipeline/features/conftest.py
mathislucka 2d7c33d
Update releasenotes/notes/feat-async-pipeline-338856a142e1318c.yaml
Amnah199 f56186c
fix: indentation, do not close loop
mathislucka 57b1cac
fix: use asyncio.run
mathislucka 47f419c
fix: format
mathislucka a326a7b
Merge branch 'main' into feat/async_pipeline
mathislucka File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,8 @@ | ||
--- | ||
highlights: > | ||
We are introducing the `AsyncPipeline`: Supports running pipelines asynchronously. Schedules components concurrently | ||
whenever possible. Leads to major speed improvements for any pipelines that may run workloads in parallel. | ||
features: | ||
- | | ||
Added a new `AsyncPipeline` implementation that allows pipelines to be executed from async code, | ||
supporting concurrent scheduling of pipeline components for faster processing. |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(nit) suggestion: keeping these imports ordered alphabetically helps locate something as the list grows