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.

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

Don't call MPI_Comm_split if NCCL_TESTS_SPLIT_MASK is not set #151

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Commits on Jun 30, 2023

  1. Don't call MPI_Comm_split if NCCL_TESTS_SPLIT_MASK is not set

    No need to call MPI_Comm_split if the NCCL_TESTS_SPLIT_MASK variable
    is not set. It only causes unnecessary traffic with no real effect,
    and in some cases might cause the ranks be invalid.
    
    Signed-off-by: Tadeusz Struk <tstruk@gigaio.com>
    tstruk committed Jun 30, 2023
    Configuration menu
    Copy the full SHA
    b322eab View commit details
    Browse the repository at this point in the history