-
Notifications
You must be signed in to change notification settings - Fork 6
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 nvidia container toolkit to Dockerfile #358
base: main
Are you sure you want to change the base?
Conversation
…o test Mujoco rendering performance with Nvidia GPU inside Docker container
07222ab
to
f3bf74a
Compare
Since I have not had much luck getting this configured, I tried a simpler workflow (no CUDA, just nvidia drivers) and got the render speed up I wanted via https://github.com/PickNikRobotics/moveit_studio_ur_ws/tree/pr-add-dockerfile-with-nvidia-drivers Unfortunately, I do not think we can just merge the nvidia-driver change into the main Dockerfile. Ironically, not because it will break things for non-nvidia users (I used the image built with nvidia drivers on an Intel integrated graphics laptop successfully), but because it will cause a segfault when launching Pro if the naive version installed by the Dockerfile (e.g. 555) does not match the version on your system (e.g. 535). Let's talk about how we want to handle this - I don't love having to keep two Dockerfiles up to date. Perhaps initially, unless there is something simple I missed with nvidia-toolkit, we could just add the commented out entries for nvidia-driver installation to the primary Dockerfile with instructions in the Quick Start to uncomment the lines and sync the driver version if you have an Nvidia card and want high render rates in the Pro sim. |
@@ -49,6 +49,25 @@ RUN --mount=type=cache,target=/var/cache/apt,sharing=locked \ | |||
/home/${USERNAME}/.ros && \ | |||
chown -R $USER_UID:$USER_GID /home/${USERNAME} /opt/overlay_ws/ | |||
|
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.
For this image we also need to install the nvidia-container-toolkit:
RUN curl -fsSL https://nvidia.github.io/libnvidia-container/gpgkey | sudo gpg --dearmor -o /usr/share/keyrings/nvidia-container-toolkit-keyring.gpg \ | |
&& curl -s -L https://nvidia.github.io/libnvidia-container/stable/deb/nvidia-container-toolkit.list | \ | |
sed 's#deb https://#deb [signed-by=/usr/share/keyrings/nvidia-container-toolkit-keyring.gpg] https://#g' | \ | |
sudo tee /etc/apt/sources.list.d/nvidia-container-toolkit.list | |
# hadolint ignore=DL3008 | ||
RUN --mount=type=cache,target=/var/cache/apt,sharing=locked \ | ||
--mount=type=cache,target=/var/lib/apt,sharing=locked \ | ||
apt-get update && apt-get install --no-install-recommends -y software-properties-common wget |
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.
apt-get update && apt-get install --no-install-recommends -y software-properties-common wget | |
apt-get update && apt-get install --no-install-recommends -y software-properties-common wget nvidia-container-toolkit |
Unsure about how general of a fix it is, but I was able to help another customer get much better performance just by changing |
That is very interesting. Yeah I think the solution to this is multistepped. We probably need to provide a /etc/docker/daemon.json configuring the nvidia and cri-o options needed, in addition to the runtime in the docker compose as Ezra just mentioned. (nvidia-ctk can do some of this) Overall I think the solution to this is to have:
|
Here is an example of my ZFS + cuda enabled docker daemon.json:
|
Mujoco camera performance with an Nvidia GPU inside a Docker container is really bad. I am hoping this will fix it. Taken from Chance's initial work.