Conversation
Signed-off-by: Nic Ma <[email protected]>
80ad38e to
8e07681
Compare
|
Hi @IsaacYangSLA and @wyli , I am trying to add distributed tests based on Horovod environment, but I got a question here: Thanks in advance. |
|
/black |
Signed-off-by: monai-bot <[email protected]>
Signed-off-by: Nic Ma <[email protected]>
Signed-off-by: Nic Ma <[email protected]>
|
Hi @wyli @IsaacYangSLA , or maybe let's just put this Horovod test in the tests to manually run locally first, then try to integrate into CI later when we have more Horovod tests? Just like what we did for the PyTorch distributed tests before? Thanks. |
|
To run this new Horovod test locally, just follow:
Thanks. |
|
@IsaacYangSLA could you please help review? |
Fixes #2507 .
Description
This PR added the Horovod unit tests for distributed data parallel.
Status
Ready
Types of changes
./runtests.sh -f -u --net --coverage../runtests.sh --quick --unittests.make htmlcommand in thedocs/folder.