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.

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

Optimize Pipeline.run() #7614

Closed
silvanocerza opened this issue Apr 29, 2024 · 0 comments · Fixed by #8019
Closed

Optimize Pipeline.run() #7614

silvanocerza opened this issue Apr 29, 2024 · 0 comments · Fixed by #8019
Assignees
Labels
2.x Related to Haystack v2.0 P1 High priority, add to the next sprint 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
@shadeMe shadeMe added P1 High priority, add to the next sprint and removed P2 Medium priority, add to the next sprint if no P1 available labels Jun 10, 2024
@shadeMe shadeMe assigned shadeMe and silvanocerza and unassigned shadeMe Jun 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2.x Related to Haystack v2.0 P1 High priority, add to the next sprint topic:core
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants