-
Notifications
You must be signed in to change notification settings - Fork 811
Change GPU check #19
Comments
Change GPU check to CUDA SDK check. Closes issue #19
Yes this seems like a reasonable alternative check. Thanks for the commit and producing Docker images! |
Just following up since I've found time to run some models. The CUDA version fails on Kepler GPUs (as expected) but works beautifully on Maxwell GPUs! |
Great to hear! As a suggestion you may want to rename the docker image to make it clear that it is Maxwell only since you're using the nervanagpu backend. In fact it would probably good to have another docker image based off our cudanet GPU backend since this will work on Kepler cards too. |
Good point - I've now replaced the CUDA version with nervanagpu-neon and cudanet-neon. As far as I can tell it isn't necessary to install cuda-convnet2 beforehand as the neon installer does it. One weird issue I've had on doing local builds is that when attempting to do a multithreaded make on the cudanet repo it works fine, but it consistently fails when run on the neon repo with the cudanet flag. I'm not going to raise an issue but if you wish to investigate further the most pertinent log I saw was
which looks like somewhere along the line pip isn't handling memory well. |
I've been making Docker images for deep learning libraries, with CPU-only and CUDA-enabled versions. I've successfully made the base version of neon , but my CUDA version doesn't get built properly. The issue in the Makefile is that
nvidia-smi
is used to check that a CUDA-enabled GPU is available, and whilst this isn't available in Docker build environments it has been possible to build libraries with the installed SDK. If a GPU isn't needed for the build, is it possible to change this test to check for something that is e.g.nvcc --version
?The text was updated successfully, but these errors were encountered: