Skip to content

PyTorch 1.4.0 : Added ibverbs dependencies via constants. #10540

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
Merged
boegel merged 4 commits into from
May 19, 2020
Merged

PyTorch 1.4.0 : Added ibverbs dependencies via constants. #10540

boegel merged 4 commits into from
May 19, 2020

Conversation

@terjekv
Copy link
Collaborator

@terjekv terjekv commented Apr 29, 2020 •
edited by boegel
Loading

(created using eb --new-pr)

Requires easybuilders/easybuild-framework#3309

@terjekv terjekv mentioned this pull request Apr 29, 2020
Closed
@terjekv terjekv changed the title Added ibverbs dependencies. Requires https://github.com/easybuilders/easybuild-framework/pull/3309 PyTorch 1.4.0 : Added ibverbs dependencies via constants. Apr 29, 2020
@boegel boegel added the bug fix label Apr 30, 2020
@boegel boegel added this to the next release (4.2.1?) milestone Apr 30, 2020
@smoors
Copy link
Contributor

smoors commented May 19, 2020

Test report by @smoors
FAILED
Build succeeded for 0 out of 2 (2 easyconfigs in this PR)
node314.hydra.os - Linux centos linux 7.7.1908, x86_64, Intel(R) Xeon(R) Gold 6148 CPU @ 2.40GHz (skylake_avx512), Python 2.7.5
See https://gist.github.com/79e3b231b7378e95d8356403416ca668 for a full test report.

@smoors
Copy link
Contributor

smoors commented May 19, 2020

(out of memory)

@terjekv
Copy link
Collaborator Author

terjekv commented May 19, 2020

(out of memory)

Ah, the joys of machine learning.

@easybuilders easybuilders deleted a comment from boegelbot May 19, 2020
@easybuilders easybuilders deleted a comment from boegelbot May 19, 2020
boegel
boegel approved these changes May 19, 2020
View reviewed changes
Copy link
Member

@boegel boegel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@boegel
Copy link
Member

boegel commented May 19, 2020

Test report by @boegel
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in this PR)
node3300.joltik.os - Linux centos linux 7.7.1908, x86_64, Intel(R) Xeon(R) Gold 6242 CPU @ 2.80GHz (cascadelake), Python 3.6.8
See https://gist.github.com/be27e1367f98fb94826a81d2c5619b64 for a full test report.

@boegel
Copy link
Member

boegel commented May 19, 2020

Going in, thanks @terjekv!

@boegel boegel merged commit 9ee1989 into easybuilders:develop May 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@boegel boegel boegel approved these changes

Assignees

No one assigned

Labels

bug fix

Projects

None yet

Milestone

4.2.1

Development

Successfully merging this pull request may close these issues.

3 participants

@terjekv @smoors @boegel