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

Optimize Pipeline.run() #7614

Open
silvanocerza opened this issue Apr 29, 2024 · 0 comments
Open

Optimize Pipeline.run() #7614

silvanocerza opened this issue Apr 29, 2024 · 0 comments
Labels
2.x Related to Haystack v2.0 P2 Medium priority, add to the next sprint if no P1 available topic:core

Comments

@silvanocerza
Copy link
Contributor

The current logic of Pipeline.run() was kinda rewritten in a haste so we cut some corners and didn't optimize it.

The code is also not well documented making it harder to understand and reason about for other devs.

Ideally this should be done after #7611 to prevent the introduction of any bugs.

This will also make it easier to add new functionalities in the future.
@silvanocerza silvanocerza added P2 Medium priority, add to the next sprint if no P1 available 2.x Related to Haystack v2.0 topic:core labels Apr 29, 2024
@silvanocerza silvanocerza self-assigned this Apr 29, 2024
@masci masci mentioned this issue May 6, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
2.x Related to Haystack v2.0 P2 Medium priority, add to the next sprint if no P1 available topic:core
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@silvanocerza