Join GitHub today
GitHub is home to over 20 million developers working together to host and review code, manage projects, and build software together.
Fix MultiGPU Testing #2931
+13
−2
Conversation
|
@thatguymike This should fix the original issue in #2926, although not for the case in #2926 (comment) |
ronghanghu
added the
ready for review
label
Aug 15, 2015
|
Setting CUDA_VISIBLE_DEVICES may be a better idea, but I didn't figure out how to set it up automatically, to ensure Multi-GPU is tested by default on multi-gpu machines, while avoiding the crash in #2926 (comment) |
ronghanghu
added a commit
that referenced
this pull request
Aug 15, 2015
|
|
ronghanghu |
25b4920
|
ronghanghu
merged commit 25b4920
into
BVLC:master
Aug 15, 2015
1 check passed
continuous-integration/travis-ci/pr
The Travis CI build passed
Details
ronghanghu
referenced
this pull request
Aug 15, 2015
Closed
runtest crash with TEST_GPUID != 0 #2926
ronghanghu
deleted the
ronghanghu:fix-gpu-test branch
Aug 15, 2015
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
ronghanghu commentedAug 15, 2015
This is a patch for multi-gpu testing issue (#2926). The problem is when calling
make runtestwith TEST_GPUID != 0 on a MultiGPU machine, solver tests will crash because gpu ids in multi-gpu tests doesn't match that of single GPU test.Note: this still doesn't fix the crash if you have 2 widely different device at the same time. #2926 (comment)