Skip to content
This repository has been archived by the owner on Jan 22, 2024. It is now read-only.

Implement cuDNN, Caffe(dev), Digits(dev) Dockerfiles for CentOS 7 #55

Closed
wants to merge 5 commits into from

Conversation

edsonke
Copy link
Contributor

@edsonke edsonke commented Feb 26, 2016

Hi, I've implemented some docker files which will help user using Digits under CentOS 7, but still need your kindly review to check these commits.
User can use DIGITS dev-server located in /tmp with these commits.

Following are some more detail about these commits

  1. I install the cuDNN library with Ubuntu deb files. As far as I know, user have to register to Nvidia forum to get .rpm , hence I did some trick(transform .deb files to .rpm using ruby-fpm) to install Ubuntu deb files instead of asking user to get the files.
  2. I build Caffe and DIGITS with these Dockerfiles instead of installing the .deb files provided by Nvidia directly. There are reasons to do this. First, both Caffe and DIGITS depends on many stuffs which might not as same as Ubuntu and their scripts inside .deb installation are incompatible to CentOS totally.

@flx42
Copy link
Member

flx42 commented Feb 26, 2016

@edsonke
Copy link
Contributor Author

edsonke commented Feb 26, 2016

Hi @flx42 ,
Appreciate for your comment, it has been updated as your recommendation.

BR

@flx42
Copy link
Member

flx42 commented Mar 7, 2016

Sorry, I don't have a lot of time to review this PR.
Quick question for now: why do you need DIGITS on CentOS?

For CUDA and Caffe I can understand, since they can be used as base image for other projects. But for DIGITS I'm not sure I see the use case.

@edsonke
Copy link
Contributor Author

edsonke commented Mar 14, 2016

Hi @flx42 ,
Sorry for late reply. I think there are at least two reasons to have these stuff:

  • For enterprise version just starting from CenOS widerspread.
  • And another reason for NV they already think DIGITS could be inference engine not only training such that docker support for CentOS is coming to confront it.

Best reagards

@flx42 flx42 closed this in c5fe8e4 Apr 15, 2016
@flx42
Copy link
Member

flx42 commented Apr 15, 2016

@edsonke sorry for letting the PR rot! I merged your cuDNN patch and the missing packages.
We decided to not include CentOS versions of Caffe and DIGITS for now, but feel free to continue maintaining your own versions using the new images.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants