Skip to content

use NCCL for GPU ops in Horovod 0.18.2 easyconfig#9562

Merged
smoors merged 2 commits intoeasybuilders:developfrom
Flamefire:20191220134315_new_pr_Horovod0182
Dec 21, 2019
Merged

use NCCL for GPU ops in Horovod 0.18.2 easyconfig#9562
smoors merged 2 commits intoeasybuilders:developfrom
Flamefire:20191220134315_new_pr_Horovod0182

Conversation

@Flamefire
Copy link
Copy Markdown
Contributor

(created using eb --new-pr)

@Flamefire Flamefire force-pushed the 20191220134315_new_pr_Horovod0182 branch from 070c069 to b5f9609 Compare December 20, 2019 15:08
@smoors
Copy link
Copy Markdown
Contributor

smoors commented Dec 20, 2019

Test report by @smoors
SUCCESS
Build succeeded for 6 out of 6 (1 easyconfigs in this PR)
node372.hydra.os - Linux centos linux 7.6.1810, Intel(R) Xeon(R) Gold 6148 CPU @ 2.40GHz, Python 2.7.5
See https://gist.github.com/cd4c707afa90bec423345ce498763a85 for a full test report.

@easybuilders easybuilders deleted a comment from boegelbot Dec 21, 2019
@easybuilders easybuilders deleted a comment from boegelbot Dec 21, 2019
@boegel boegel changed the title [Horovod] Use NCCL for GPU ops use NCCL for GPU ops in Horovod 0.18.2 easyconfig Dec 21, 2019
@boegel boegel added this to the release after 4.1.0 (4.1.1?) milestone Dec 21, 2019
@boegel
Copy link
Copy Markdown
Member

boegel commented Dec 21, 2019

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
generoso - Linux centos linux 7.6.1810, Intel(R) Xeon(R) CPU E5-2660 v4 @ 2.00GHz, Python 3.6.8
See https://gist.github.com/3eee812b35d01fc6a2db35bfd483d865 for a full test report.

Copy link
Copy Markdown
Member

@boegel boegel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@smoors
Copy link
Copy Markdown
Contributor

smoors commented Dec 21, 2019

Going in, thanks @Flamefire!

@smoors smoors merged commit 2618e6f into easybuilders:develop Dec 21, 2019
@Flamefire Flamefire deleted the 20191220134315_new_pr_Horovod0182 branch December 21, 2019 20:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants