Skip to content
This repository has been archived by the owner on Nov 17, 2023. It is now read-only.

Sync Horovod distributed training examples with latest changes #14748

Merged
merged 2 commits into from
May 17, 2019

Conversation

yuxihu
Copy link
Member

@yuxihu yuxihu commented Apr 19, 2019

Sync distributed training examples with MXNet and Horovod with latest changes in Horovod:

  1. Added DistributedTrainer, a subclass of Gluon Trainer in Horovod, to make distributed training using MXNet Gluon API with PS and Horovod have the same internal workflow.
  2. Resovled the known GCC incompatibility issue.

@yuxihu yuxihu requested a review from szha as a code owner April 19, 2019 22:02
@yuxihu
Copy link
Member Author

yuxihu commented Apr 19, 2019

@mxnet-label-bot add [pr-work-in-progress]

@marcoabreu marcoabreu added the pr-work-in-progress PR is still work in progress label Apr 19, 2019
@vandanavk
Copy link
Contributor

@mxnet-label-bot add [Distributed]

@yuxihu
Copy link
Member Author

yuxihu commented May 14, 2019

@apeforest @eric-haibin-lin please help review.

@mxnet-label-bot update [Distributed, pr-awaiting-review]

@marcoabreu marcoabreu added pr-awaiting-review PR is waiting for code review Distributed and removed Distributed pr-work-in-progress PR is still work in progress labels May 14, 2019
@yuxihu
Copy link
Member Author

yuxihu commented May 17, 2019

@mxnet-label-bot update [Distributed, pr-awaiting-merge]

@marcoabreu marcoabreu added pr-awaiting-merge Review and CI is complete. Ready to Merge and removed pr-awaiting-review PR is waiting for code review labels May 17, 2019
@apeforest apeforest merged commit 5b4e866 into apache:master May 17, 2019
@yuxihu yuxihu deleted the hvd_example branch June 12, 2019 16:16
haohuanw pushed a commit to haohuanw/incubator-mxnet that referenced this pull request Jun 23, 2019
…e#14748)

* sync examples with latest changes in Horovod

* update README
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Distributed pr-awaiting-merge Review and CI is complete. Ready to Merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants