This repository has been archived by the owner on Nov 3, 2023. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 34
PTL 1.2 Compatibility #15
Merged
Merged
Changes from all commits
Commits
Show all changes
27 commits
Select commit
Hold shift + click to select a range
ba5492d
wip
amogkam 0071abc
update ddp
amogkam 3e389cb
Merge branch 'main' into 1.2
amogkam 504e731
remove accelerator
amogkam fb3e273
re-enable tests
amogkam c2f467c
re-enable
amogkam e9ea86b
unpin ptl
amogkam 9a8b2b3
updates
amogkam b29123f
update gpu tests
amogkam 6d848b5
install ptl master
amogkam 8fff04b
update horovod
amogkam 7b6d112
update horovod example
amogkam 3488ff6
skip failing mnist examples
amogkam 05d805a
update readme
amogkam aa0761a
formatting
amogkam 7580b27
formatting
amogkam 17ff287
fix
amogkam 19f0957
update tune tests
amogkam 1e1a6f9
skip more mnist examples
amogkam e04aed3
lint
amogkam b80421b
fix
amogkam ee1e551
disable ptl checkpointing for now
amogkam e83ef44
fix
amogkam 3bdc9fb
use 1.2.4
amogkam e40b226
Update ray_lightning/ray_ddp.py
amogkam 3fc84b2
comments
amogkam eb1e851
Merge branch '1.2' of github.com:amogkam/ray_lightning_accelerators i…
amogkam File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -47,34 +47,34 @@ jobs: | |
python -m pytest -v --durations=0 -x test_horovod.py | ||
python -m pytest -v --durations=0 -x test_tune.py | ||
|
||
# test_linux_ray_master_examples: | ||
# runs-on: ubuntu-latest | ||
# timeout-minutes: 12 | ||
# steps: | ||
# - uses: actions/checkout@v2 | ||
# - name: Set up Python 3.7 | ||
# uses: actions/setup-python@v2 | ||
# with: | ||
# python-version: 3.7 | ||
# - name: Install dependencies | ||
# run: | | ||
# python -m pip install --upgrade pip | ||
# python -m pip install --upgrade setuptools | ||
# python -m pip install codecov | ||
# python -m pip install -U https://s3-us-west-2.amazonaws.com/ray-wheels/latest/ray-2.0.0.dev0-cp37-cp37m-manylinux2014_x86_64.whl | ||
# if [ -f requirements-test.txt ]; then python -m pip install -r requirements-test.txt; fi | ||
# HOROVOD_WITH_GLOO=1 HOROVOD_WITHOUT_MPI=1 HOROVOD_WITHOUT_MXNET=1 pip install git+https://github.com/horovod/horovod.git | ||
# - name: Install package | ||
# run: | | ||
# python -m pip install -e . | ||
# - name: Run Examples | ||
# run: | | ||
# pushd examples/ | ||
# echo "running ray_ddp_example.py" && python ray_ddp_example.py --smoke-test | ||
# echo "running ray_ddp_example.py with Tune" && python ray_ddp_example.py --smoke-test --tune | ||
# echo "running ray_ddp_tune.py" && python ray_ddp_tune.py --smoke-test | ||
# echo "running ray_horovod_example.py" && python ray_horovod_example.py --smoke-test | ||
# echo "running ray_horovod_example.py with Tune" && python ray_horovod_example.py --smoke-test --tune | ||
test_linux_ray_master_examples: | ||
runs-on: ubuntu-latest | ||
timeout-minutes: 12 | ||
steps: | ||
- uses: actions/checkout@v2 | ||
- name: Set up Python 3.7 | ||
uses: actions/setup-python@v2 | ||
with: | ||
python-version: 3.7 | ||
- name: Install dependencies | ||
run: | | ||
python -m pip install --upgrade pip | ||
python -m pip install --upgrade setuptools | ||
python -m pip install codecov | ||
python -m pip install -U https://s3-us-west-2.amazonaws.com/ray-wheels/latest/ray-2.0.0.dev0-cp37-cp37m-manylinux2014_x86_64.whl | ||
if [ -f requirements-test.txt ]; then python -m pip install -r requirements-test.txt; fi | ||
HOROVOD_WITH_GLOO=1 HOROVOD_WITHOUT_MPI=1 HOROVOD_WITHOUT_MXNET=1 pip install git+https://github.com/horovod/horovod.git | ||
- name: Install package | ||
run: | | ||
python -m pip install -e . | ||
- name: Run Examples | ||
run: | | ||
pushd examples/ | ||
# echo "running ray_ddp_example.py" && python ray_ddp_example.py --smoke-test | ||
# echo "running ray_ddp_example.py with Tune" && python ray_ddp_example.py --smoke-test --tune | ||
# echo "running ray_ddp_tune.py" && python ray_ddp_tune.py --smoke-test | ||
# echo "running ray_horovod_example.py" && python ray_horovod_example.py --smoke-test | ||
# echo "running ray_horovod_example.py with Tune" && python ray_horovod_example.py --smoke-test --tune | ||
|
||
test_linux_ray_release: | ||
runs-on: ubuntu-latest | ||
|
@@ -104,31 +104,31 @@ jobs: | |
python -m pytest -v --durations=0 -x test_tune.py | ||
|
||
|
||
# test_linux_ray_release_examples: | ||
# runs-on: ubuntu-latest | ||
# timeout-minutes: 12 | ||
# steps: | ||
# - uses: actions/checkout@v2 | ||
# - name: Set up Python 3.7 | ||
# uses: actions/setup-python@v2 | ||
# with: | ||
# python-version: 3.7 | ||
# - name: Install dependencies | ||
# run: | | ||
# python -m pip install --upgrade pip | ||
# python -m pip install --upgrade setuptools | ||
# python -m pip install codecov | ||
# python -m pip install -U ray | ||
# if [ -f requirements-test.txt ]; then python -m pip install -r requirements-test.txt; fi | ||
# HOROVOD_WITH_GLOO=1 HOROVOD_WITHOUT_MPI=1 HOROVOD_WITHOUT_MXNET=1 pip install -U git+https://github.com/horovod/horovod.git | ||
# - name: Install package | ||
# run: | | ||
# python -m pip install -e . | ||
# - name: Run Examples | ||
# run: | | ||
# pushd examples/ | ||
# echo "running ray_ddp_example.py" && python ray_ddp_example.py --smoke-test | ||
# echo "running ray_ddp_example.py with Tune" && python ray_ddp_example.py --smoke-test --tune | ||
# echo "running ray_ddp_tune.py" && python ray_ddp_tune.py --smoke-test | ||
# echo "running ray_horovod_example.py" && python ray_horovod_example.py --smoke-test | ||
# echo "running ray_horovod_example.py with Tune" && python ray_horovod_example.py --smoke-test --tune | ||
test_linux_ray_release_examples: | ||
runs-on: ubuntu-latest | ||
timeout-minutes: 12 | ||
steps: | ||
- uses: actions/checkout@v2 | ||
- name: Set up Python 3.7 | ||
uses: actions/setup-python@v2 | ||
with: | ||
python-version: 3.7 | ||
- name: Install dependencies | ||
run: | | ||
python -m pip install --upgrade pip | ||
python -m pip install --upgrade setuptools | ||
python -m pip install codecov | ||
python -m pip install -U ray | ||
if [ -f requirements-test.txt ]; then python -m pip install -r requirements-test.txt; fi | ||
HOROVOD_WITH_GLOO=1 HOROVOD_WITHOUT_MPI=1 HOROVOD_WITHOUT_MXNET=1 pip install -U git+https://github.com/horovod/horovod.git | ||
- name: Install package | ||
run: | | ||
python -m pip install -e . | ||
- name: Run Examples | ||
run: | | ||
pushd examples/ | ||
# echo "running ray_ddp_example.py" && python ray_ddp_example.py --smoke-test | ||
# echo "running ray_ddp_example.py with Tune" && python ray_ddp_example.py --smoke-test --tune | ||
# echo "running ray_ddp_tune.py" && python ray_ddp_tune.py --smoke-test | ||
# echo "running ray_horovod_example.py" && python ray_horovod_example.py --smoke-test | ||
# echo "running ray_horovod_example.py with Tune" && python ray_horovod_example.py --smoke-test --tune | ||
Comment on lines
+130
to
+134
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. do we plan to not run any? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. See above comment. |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1,4 @@ | ||
from ray_lightning.ray_ddp import RayAccelerator | ||
from ray_lightning.ray_horovod import HorovodRayAccelerator | ||
from ray_lightning.ray_ddp import RayPlugin | ||
from ray_lightning.ray_horovod import HorovodRayPlugin | ||
|
||
__all__ = ["RayAccelerator", "HorovodRayAccelerator"] | ||
__all__ = ["RayPlugin", "HorovodRayPlugin"] |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do we plan to not run any of these?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These all use the MNIST dataset from torchvision which is failing right now due to this error https://discuss.pytorch.org/t/mnist-server-down/114433. After the next torchvision release we can re-enable these tests (and the ones on the Ray repo).