Skip to content

Combine forward and backward dependencies#6

Merged
angelhof merged 13 commits intomainfrom
forward_backward_combination
Feb 27, 2023
Merged

Combine forward and backward dependencies#6
angelhof merged 13 commits intomainfrom
forward_backward_combination

Conversation

@angelhof
Copy link
Copy Markdown
Member

@angelhof angelhof commented Dec 8, 2022

TODOS:

  • Run non-first commands in sandbox (modify executor.py)
  • Run all commands asynchronously (modify orch.py)
  • Check dependencies and decide whether to commit sandboxes or not (probably modify orch.py)

@angelhof
Copy link
Copy Markdown
Member Author

angelhof commented Dec 9, 2022

@gliargovas the final TODO is described in a comment here (https://github.com/binpash/dynamic-parallelizer/blob/forward_backward_combination/overlay-sandbox/run-sandboxed.sh#L42). We essentially need to do the sandbox-file-change check inside orch, and only commit the write changes in the upperdir of each sandbox if they do not interfere with the changes of the non sandboxed command.

@angelhof angelhof merged commit 4f109e1 into main Feb 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants