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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Beaker Executor should execute multiple Tango steps in one Beaker experiment #457

Open
dirkgr opened this issue Oct 20, 2022 · 3 comments
Open
Labels
feature request integration: beaker Relates to the Beaker integration

Comments

@dirkgr
Copy link
Member

dirkgr commented Oct 20, 2022

馃殌 The feature, motivation and pitch

Beaker experiments have significant overhead. When we're running many small Tango steps, we can save some time by running multiple steps at once.

An issue is that we don't know ahead of time how long a step will take, so we don't know which steps make sense to group together. When running like this, it would make sense for Beaker experiments to poll the workspace for new work. This would be a drastic change in how we execute things, and it would overlap with Beaker functionality in uncomfortable ways. This feature requires some careful thought.

Alternatives

No response

Additional context

No response

@dirkgr
Copy link
Member Author

dirkgr commented Oct 20, 2022

The caveats in the description of this issue stand, but we can probably find sub-graphs of the whole execution graph that always make sense to group together into one Beaker experiment. A run of steps where each one depends on the previous step, and no other dependencies are unfulfilled, those could always be grouped. It would cut the overhead of my current experiment by 66%.

@epwalsh
Copy link
Member

epwalsh commented Oct 20, 2022

A run of steps where each one depends on the previous step, and no other dependencies are unfulfilled, those could always be grouped. It would cut the overhead of my current experiment by 66%.

That was my thought as well. These steps would also have to have matching step resource requirements.

@dirkgr
Copy link
Member Author

dirkgr commented Oct 20, 2022

These steps would also have to have matching step resource requirements.

Or we take the max of the resource requirements.

@epwalsh epwalsh added the integration: beaker Relates to the Beaker integration label Nov 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request integration: beaker Relates to the Beaker integration
Projects
None yet
Development

No branches or pull requests

2 participants