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

[backport] Avoid unload module call in PureNcclCommunicator #7787

Merged
merged 1 commit into from Jul 23, 2019
Merged

[backport] Avoid unload module call in PureNcclCommunicator #7787

merged 1 commit into from Jul 23, 2019

Conversation

shu65
Copy link
Member

@shu65 shu65 commented Jul 22, 2019

Backport of #7600

@shu65
Copy link
Member Author

shu65 commented Jul 22, 2019

Jenkins, test this please.

@shu65 shu65 added ChainerMN Related to ChainerMN. backport Pull request that is backported from a more recent version. cat:bug Bug report or fix. labels Jul 22, 2019
@chainer-ci
Copy link
Member

Jenkins CI test (for commit 39e057a, target branch v6) succeeded!

@kuenishi kuenishi self-requested a review July 22, 2019 23:50
@kuenishi
Copy link
Member

I've observed several integration test failures, although they doesn't seem directly related to this change.

@kuenishi kuenishi merged commit f6fbbfc into chainer:v6 Jul 23, 2019
@kuenishi kuenishi added this to the v6.3.0 milestone Jul 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kuenishi kuenishi Awaiting requested review from kuenishi

Assignees
No one assigned
Labels
backport Pull request that is backported from a more recent version. cat:bug Bug report or fix. ChainerMN Related to ChainerMN.
Projects
None yet
Milestone
v6.3.0
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@shu65 @chainer-ci @kuenishi