-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Labels
Comments
This was referenced Jun 10, 2024
This was referenced Jun 19, 2024
This was referenced Jul 4, 2024
This was referenced Jul 11, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
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.
The text was updated successfully, but these errors were encountered: