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

exp: refactor workspace execution into an executor #6847

Open
pmrowla opened this issue Oct 21, 2021 · 1 comment
Open

exp: refactor workspace execution into an executor #6847

pmrowla opened this issue Oct 21, 2021 · 1 comment
Labels
A: executors Related to the executors feature p2-medium Medium priority, should be done, but less important refactoring Factoring and re-factoring

Comments

@pmrowla
Copy link
Contributor

pmrowla commented Oct 21, 2021

Somewhat adjacent to #6440 (but not a direct requirement)

Currently workspace execution for experiments is handled directly in repo.experiments, while temp dir (and eventually ssh) execution are handled through the experiments.executor classes. While the executor classes are being reorganized to account for ssh/remote executors, it makes sense for us to refactor the workspace execution handling so that it is in line with the rest of the API.
@pmrowla pmrowla added refactoring Factoring and re-factoring p2-medium Medium priority, should be done, but less important A: executors Related to the executors feature labels Oct 21, 2021
@pmrowla
Copy link
Contributor Author

pmrowla commented Oct 21, 2021

This is something I'll probably get to once the basic remote execution scenarios are taken care of.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A: executors Related to the executors feature p2-medium Medium priority, should be done, but less important refactoring Factoring and re-factoring
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@pmrowla