Open Michaelvll opened 1 week ago
Thought: should we make it robust and then just turn it on by default? Reducing flags to learn.
On Tue, Nov 12, 2024 at 11:56 Zhanghao Wu @.***> wrote:
TODO:
- Add --fast in the managed jobs / managed spot pages.
Tested (run the relevant ones):
- Code formatting: bash format.sh
- Any manual or new tests for this PR (please specify below)
- All smoke tests: pytest tests/test_smoke.py
- Relevant individual smoke tests: pytest tests/test_smoke.py::test_fill_in_the_name
- Backward compatibility tests: conda deactivate; bash -i tests/backward_compatibility_tests.sh
You can view, comment on, or merge this pull request online at:
https://github.com/skypilot-org/skypilot/pull/4339 Commit Summary
- e64f68c https://github.com/skypilot-org/skypilot/pull/4339/commits/e64f68c602dc2a47c743b76b22e72484003689dc [Docs] Use
--fast
for job submissionFile Changes
(1 file https://github.com/skypilot-org/skypilot/pull/4339/files)
- M docs/source/running-jobs/many-jobs.rst https://github.com/skypilot-org/skypilot/pull/4339/files#diff-f6a2b93f7444a8c6a5f63847c06630a52a10bbfea23784040bbd27b5aedfc3c2 (5)
Patch Links:
- https://github.com/skypilot-org/skypilot/pull/4339.patch
- https://github.com/skypilot-org/skypilot/pull/4339.diff
— Reply to this email directly, view it on GitHub https://github.com/skypilot-org/skypilot/pull/4339, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAEQWHRJ4AJBMN3DVCY3Z432AJMNZAVCNFSM6AAAAABRU27U2SVHI2DSMVQWIX3LMV43ASLTON2WKOZSGY2TGMJYGYYDENY . You are receiving this because you are subscribed to this thread.Message ID: @.***>
Thought: should we make it robust and then just turn it on by default? Reducing flags to learn.
Yes, this is an intermediate edit. We will switch it back when we have the robustness stuff merged.
TODO:
--fast
in the managed jobs / managed spot pages.Tested (run the relevant ones):
bash format.sh
pytest tests/test_smoke.py
pytest tests/test_smoke.py::test_fill_in_the_name
conda deactivate; bash -i tests/backward_compatibility_tests.sh