-
Notifications
You must be signed in to change notification settings - Fork 21.5k
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
[ROCm] use ncclAllToAll for rocm #75128
Conversation
Signed-off-by: Kyle Chen <kylechen@amd.com>
🔗 Helpful links
💊 CI failures summary and remediationsAs of commit ad30b1c (more details on the Dr. CI page): 💚 💚 Looks good so far! There are no failures yet. 💚 💚 This comment was automatically generated by Dr. CI (expand for details).Please report bugs/suggestions to the (internal) Dr. CI Users group. |
Signed-off-by: Kyle Chen <kylechen@amd.com>
@pytorchbot merge this |
Merge failed due to Matched rule superuser, but it was not reviewed yet by any of:ymao1993,zzzwen,Gamrix,cherie11,dahsh, ... |
@pytorchbot merge this |
Hey @KyleCZH. |
Summary: use ncclAllToAll for rocm version > 5.0; ROCm/rccl#503 detail on ncclAllToAll: ROCm/rccl#503 jithunnair-amd amathews-amd Pull Request resolved: #75128 Approved by: https://github.com/wenkaidu, https://github.com/yzygitzh, https://github.com/seemethere Test Plan: contbuild & OSS CI, see https://hud.pytorch.org/commit/pytorch/pytorch/97fbe6f0a4d136d34fae851ffee823cb5e73bf71 Reviewed By: seemethere, osalpekar Differential Revision: D35874469 fbshipit-source-id: 653579837396cdc55bf3a7c2be0e893e16990c9a
use ncclAllToAll for rocm version > 5.0; ROCm/rccl#503 detail on ncclAllToAll: ROCm/rccl#503 @jithunnair-amd @amathews-amd Pull Request resolved: pytorch#75128 Approved by: https://github.com/wenkaidu, https://github.com/yzygitzh, https://github.com/seemethere
use ncclAllToAll for rocm version > 5.0; ROCm/rccl#503 detail on ncclAllToAll: ROCm/rccl#503 @jithunnair-amd @amathews-amd Pull Request resolved: pytorch#75128 Approved by: https://github.com/wenkaidu, https://github.com/yzygitzh, https://github.com/seemethere
use ncclAllToAll for rocm version > 5.0; ROCm/rccl#503 detail on ncclAllToAll: ROCm/rccl#503 @jithunnair-amd @amathews-amd Pull Request resolved: pytorch#75128 Approved by: https://github.com/wenkaidu, https://github.com/yzygitzh, https://github.com/seemethere
use ncclAllToAll for rocm version > 5.0; ROCm/rccl#503 detail on ncclAllToAll: ROCm/rccl#503 @jithunnair-amd @amathews-amd Pull Request resolved: pytorch#75128 Approved by: https://github.com/wenkaidu, https://github.com/yzygitzh, https://github.com/seemethere
use ncclAllToAll for rocm version > 5.0; ROCm/rccl#503 detail on ncclAllToAll: ROCm/rccl#503 @jithunnair-amd @amathews-amd Pull Request resolved: pytorch#75128 Approved by: https://github.com/wenkaidu, https://github.com/yzygitzh, https://github.com/seemethere
use ncclAllToAll for rocm version > 5.0; ROCm/rccl#503
detail on ncclAllToAll:
ROCm/rccl#503
@jithunnair-amd @amathews-amd