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

Add wrapper functions for ncclGroupStart() and ncclGroupEnd() #742

Merged
merged 1 commit into from
Jul 19, 2022

Conversation

seunghwak
Copy link
Contributor

This PR adds group_start() and group_end() functions. These functions internally call ncclGroupStart() and ncclGroupEnd(), respectively.

@seunghwak seunghwak requested a review from a team as a code owner July 14, 2022 21:31
@github-actions github-actions bot added the cpp label Jul 14, 2022
@seunghwak seunghwak self-assigned this Jul 14, 2022
@seunghwak seunghwak added enhancement New feature or request 3 - Ready for Review improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Jul 14, 2022
@cjnolet
Copy link
Member

cjnolet commented Jul 19, 2022

@gpucibot merge

@rapids-bot rapids-bot bot merged commit fd2595c into rapidsai:branch-22.08 Jul 19, 2022
rapids-bot bot pushed a commit to rapidsai/cugraph that referenced this pull request Jul 21, 2022
#2413)

Dependent on rapidsai/raft#742

Place multiple broadcast operations inside ncclGroupStart and ncclGroupEnd wrappers to improve performance.

Authors:
  - Seunghwa Kang (https://github.com/seunghwak)

Approvers:
  - Chuck Hastings (https://github.com/ChuckHastings)

URL: #2413
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3 - Ready for Review cpp enhancement New feature or request improvement Improvement / enhancement to an existing function non-breaking Non-breaking change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants