Skip to content

remove useless definition of TORCH_CUDA_ARCH_LIST in PyTorch easyconfig that doesn't use CUDA#6719

Merged
wpoely86 merged 1 commit intoeasybuilders:developfrom
boegel:pytorch031_no_cuda
Aug 17, 2018
Merged

remove useless definition of TORCH_CUDA_ARCH_LIST in PyTorch easyconfig that doesn't use CUDA#6719
wpoely86 merged 1 commit intoeasybuilders:developfrom
boegel:pytorch031_no_cuda

Conversation

@boegel
Copy link
Copy Markdown
Member

@boegel boegel commented Aug 17, 2018

This was inherited from PyTorch-0.3.1-foss-2018a-Python-3.6.4-CUDA-9.1.85.eb via #6152, but it doesn't make any sense to have it there...

cc @wpoely86, @smoors, @zao

@boegel boegel added the bug fix label Aug 17, 2018
@boegel boegel added this to the 3.6.3 milestone Aug 17, 2018
@boegel
Copy link
Copy Markdown
Member Author

boegel commented Aug 17, 2018

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
node3126.skitty.os - Linux centos linux 7.4.1708, Intel(R) Xeon(R) Gold 6140 CPU @ 2.30GHz, Python 2.7.5
See https://gist.github.com/28d9e390170327aa1ff30c64d41c1a94 for a full test report.

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Aug 17, 2018

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
node2606.swalot.os - Linux centos linux 7.4.1708, Intel(R) Xeon(R) CPU E5-2660 v3 @ 2.60GHz, Python 2.7.5
See https://gist.github.com/0e1337cc4b111b94198de2154df7e08f for a full test report.

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Aug 17, 2018

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
node2083.delcatty.os - Linux centos linux 7.4.1708, Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz, Python 2.7.5
See https://gist.github.com/eaca0cd216d6bc1837435ea29524cb9a for a full test report.

@wpoely86 wpoely86 merged commit 0433574 into easybuilders:develop Aug 17, 2018
@boegel boegel deleted the pytorch031_no_cuda branch August 17, 2018 12:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@wpoely86 wpoely86 wpoely86 approved these changes

Assignees

No one assigned

Labels

bug fix

Projects

None yet

Milestone

3.7.0

Development

Successfully merging this pull request may close these issues.

2 participants

@boegel @wpoely86