-
Notifications
You must be signed in to change notification settings - Fork 57
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
Add Dockerfile #71
Add Dockerfile #71
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
First of all, we really appreciate your contribution.We don't have access to Docker- & GPU-enabled machine so we cannot test it right now. It may take some time until we merge your PR. I'm sorry about that.
# Using ChainerMN via Docker | ||
|
||
This directory contains a `Dockerfile` to quickly get up and running with | ||
ChainerMN via [Docker](http://www.docker.com/). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you add a note that this dockerfile does not support fast interconnect? like:
If you run ChainerMN on mutiple computing nodes, we strongly
recommend to use high speed interconnect such as Infiniband
or OmniPath to obtain maximum performance of ChainerMN.
because to obtain the best performance shown in Takuya in GTC'17, fast interconnect is a must.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Alright, note has been added
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
docker/Dockerfile
Outdated
apt-get install -y \ | ||
|
||
wget \ | ||
vim \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Well, we don’t back any group in the Editor War ;)
Hi @MannyKayy I am trying out parts of this docker file for chainercv, and
Any idea how opencv 2 should be added to the docker file? |
@pksorensen opencv is abit messy to deal with. I created a bash script that works with my personal development environment. Feel free to fork. |
Can one of the admins verify this patch? |
ChainerMN support on k8s is now ongoing. Please refer to https://chainer.org/general/2018/05/10/chainermn-on-kubernetes.html |
Dockerfile for chainermn