Commit 228fcff
(torchx/runtime/hpo) fix ax_test for fb, cleaned up apps/* and added prepend_cwd runopt to local_scheduler + some local scheduler code cleanup (#210)
Summary:
Pull Request resolved: #210
1. Makes ax_test.py work again in sandcastle
2. Clean up `apps/*` directory of unused/empty placeholder modules
3. Add `//torchx/apps:penv` python_binary to be able to deploy apps as penv
4. Add `prepend_cwd` runopt for local scheduler to prepend CWD to replica PATH so that binaries in CWD can take precedence over the ones in PATH (needed to enable ax_test in fb)
5. Some syntactical cleanup for local_scheduler
6. (minimally) deletes base_image (yes it doesn't delete all of it, I'm trying to keep the diff as relevant to enabling ax_test, but had to remove some of it to simplify local_scheduler logic)
Reviewed By: aivanou
Differential Revision: D31262463
fbshipit-source-id: 3f1649e4bc062f3fe6b54f13b03d11b53164fc8d1 parent 46e4392 commit 228fcff
File tree
18 files changed
+216
-198
lines changed- torchx
- apps
- captum
- test
- data
- test
- hpo
- test
- io
- test
- serve
- train
- test
- utils
- test
- runtime/hpo/test
- schedulers
- test
- specs
18 files changed
+216
-198
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
4 | 4 | | |
5 | 5 | | |
6 | 6 | | |
7 | | - | |
8 | | - | |
9 | | - | |
10 | | - | |
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
Whitespace-only changes.
Whitespace-only changes.
Whitespace-only changes.
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
0 commit comments