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.

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

WIP: Add CUDA and cuDNN #21

Closed
wants to merge 2 commits into from

Conversation

jakirkham
Copy link
Member

Adds a docker image with CUDA and cuDNN. This is being done for CUDA as it is difficult to redistribute correctly. As far as cuDNN, we are not allowed to redistribute it; however, we are allowed to use their docker image this includes it. More discussion can be seen in this issue ( conda-forge/staged-recipes#654 ). Please share thoughts and feedback on this potential solution.

cc @msarahan @lebedov

@msarahan
Copy link
Member

This is a good thing but I still want a rock-solid, possibly written
agreement with Nvidia on this usage.

On Sun, May 29, 2016, 18:12 jakirkham notifications@github.com wrote:

Adds a docker image with CUDA and cuDNN. This is being done for CUDA as it
is difficult to redistribute correctly. As far as cuDNN, we are not allowed
to redistribute it; however, we are allowed to use their docker image this
includes it. More discussion can be seen in this issue (
conda-forge/staged-recipes#654
conda-forge/staged-recipes#654 ). Please
share thoughts and feedback on this potential solution.

cc @msarahan https://github.com/msarahan @lebedov

https://github.com/lebedov

You can view, comment on, or merge this pull request online at:

#21
Commit Summary

  • linux-anvil-cudnn: Add image with CUDA and cuDNN.
  • circle: Add testing for the CUDA/cuDNN image.

File Changes

Patch Links:


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#21, or mute the thread
https://github.com/notifications/unsubscribe/AACV-UaQJ-lVc2vJeRTY5KJRjS0KlIJ-ks5qGh1ogaJpZM4IpZt4
.

@jakirkham
Copy link
Member Author

Sure that sounds reasonable. Though I think you are in a better position to get that sort of thing.

@kyleabeauchamp
Copy link

For the sake of my own curiosity, is there any update on the CUDA support?

@jakirkham
Copy link
Member Author

I have been meaning to raise that in a meeting for weeks. It seems that @msarahan and I keep missing each other. Also it has been lower on the stack with conda-build 2 and now Travis VM changes. Maybe try to revisit it in like a month. Will try to ping some people in the interim.

@jakirkham
Copy link
Member Author

Closing out as PR ( #93 ) supersedes this PR.

@jakirkham jakirkham closed this Apr 29, 2019
@jakirkham jakirkham deleted the add_cuda_cudnn branch April 29, 2019 19:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants