Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Add intra_rank to all communicators #177

Merged
merged 5 commits into from
Dec 25, 2017
Merged

Add intra_rank to all communicators #177

merged 5 commits into from
Dec 25, 2017

Commits on Dec 25, 2017

  1. minor fixes

    @keisukefukuda
    keisukefukuda committed Dec 25, 2017
    Configuration menu
    Copy the full SHA
    b2f2217 View commit details
    Browse the repository at this point in the history

  2. fix pep8

    @keisukefukuda
    keisukefukuda committed Dec 25, 2017
    Configuration menu
    Copy the full SHA
    2fb4302 View commit details
    Browse the repository at this point in the history

  3. fixed a unitest that used hasattr('intra_rank') to determine whether … 

    …GPU is used or not.
    @keisukefukuda
    keisukefukuda committed Dec 25, 2017
    Configuration menu
    Copy the full SHA
    0d0cbcb View commit details
    Browse the repository at this point in the history

  4. Changed PureNCCLCommunicator() to pass use_nccl=True for Communicator… 

    …Base().
    @keisukefukuda
    keisukefukuda committed Dec 25, 2017
    Configuration menu
    Copy the full SHA
    c80daba View commit details
    Browse the repository at this point in the history

  5. added tests for intra_rank and inter_rank

    @keisukefukuda
    keisukefukuda committed Dec 25, 2017
    Configuration menu
    Copy the full SHA
    1b7e4da View commit details
    Browse the repository at this point in the history