-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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.
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
[core][experimental] Add multi-GPU CI tests for accelerated DAG #45259
Merged
Merged
Changes from all commits
Commits
Show all changes
67 commits
Select commit
Hold shift + click to select a range
2d22636
TorchTensor wrappers
stephanie-wang 3a75b5a
test
stephanie-wang 0a8a8da
copy
stephanie-wang 32eaec9
update
stephanie-wang 0f8d092
torch device
stephanie-wang e067f5f
errors
stephanie-wang e0774b8
test
stephanie-wang d065935
GPU
f0813c4
temp benchmark
5fb4166
with_type_hint
stephanie-wang 84fe2c0
skip GPU tests
stephanie-wang 6375fd0
Merge remote-tracking branch 'upstream/master' into dag-gpu-channels
stephanie-wang 5bab410
clean
stephanie-wang 611c6aa
init nccl group
3f16871
NCCL channel
b917eb8
NCCL group works
fea789b
micro
388621d
update
stephanie-wang c35b3bb
Merge remote-tracking branch 'upstream/master' into dag-gpu-channels
stephanie-wang 3575431
torch
stephanie-wang e667dae
TODO
fb93b06
TODO
4fb6897
Merge branch 'master' into dag-gpu-channels
stephanie-wang 7334e8a
typing
stephanie-wang fd59817
Merge branch 'dag-gpu-channels' of github.com:stephanie-wang/ray into…
stephanie-wang 2788aa3
fix deadlock on shutdown
cd8ae72
Merge remote-tracking branch 'origin/dag-gpu-channels' into dag-nccl
601e600
files
649cbea
missing files
3d39945
move
stephanie-wang 0b10d32
lint
f074c29
x
c5ae955
Merge remote-tracking branch 'upstream/master' into dag-nccl
stephanie-wang fdd277f
lint
stephanie-wang e21d36b
comment
stephanie-wang 31b02ef
refactor
1b01954
doc
ef19f4e
call get_unique_id on nccl actor
69342ca
avoid nccl util import
f218a85
Update python/ray/dag/compiled_dag_node.py
stephanie-wang 8489f9b
update
3a9049d
Merge remote-tracking branch 'upstream/master' into dag-nccl
450a228
Merge branch 'dag-nccl' of github.com:stephanie-wang/ray into dag-nccl
1b7ac31
update
stephanie-wang 4a37388
Multiple GPU test
stephanie-wang acff481
multi gpu
stephanie-wang 1ca7107
CI
stephanie-wang 75b4daf
lint
stephanie-wang b5afe24
microbenchmark
stephanie-wang c208108
x
stephanie-wang 97064af
core GPU build
stephanie-wang ed5e907
remove multi-GPU CI tests
stephanie-wang ba38c92
test
4e9df06
Merge branch 'dag-nccl' of github.com:stephanie-wang/ray into dag-nccl
stephanie-wang 2f7c78e
doc
stephanie-wang c27d4db
Revert "remove multi-GPU CI tests"
stephanie-wang b1ce16d
Merge remote-tracking branch 'upstream/master' into dag-nccl
stephanie-wang c2017be
CI
stephanie-wang c8599cb
CUDA_VISIBLE_DEVICES?
stephanie-wang 96082dc
debug
stephanie-wang 8ddee51
fix
stephanie-wang e4979e4
debug
stephanie-wang d1806c1
fix
stephanie-wang 997fd8e
fix build?
stephanie-wang 5dab532
build
stephanie-wang 7ec0dc6
fix?
stephanie-wang 91e8a4b
remove debug
stephanie-wang 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
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
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.
add
|| true
, since the test fails it won't reach that sleep statement