nvidia-patch icon indicating copy to clipboard operation
nvidia-patch copied to clipboard

Modify Docker section in Readme

Open regbo opened this issue 2 years ago • 1 comments

Hello,

I should have known this from the start, but the readme on this repo is unclear when it comes to docker. I read the docker section as a requirement, not as an alternative, if this patch can't be installed on the host. I would suggest changing:

It is possible to use this patch with nvidia-docker containers, even if host machine hasn't patched drivers. See Dockerfile for example.

to

If you are unable to install this patch on a host machine, it is possible to use this patch within nvidia-docker containers. See Dockerfile for example.

regbo avatar Nov 28 '22 14:11 regbo

I completely agree. It was only after I stumbled upon this that I realized I had to install the patch solely on the host for it to work inside docker. Can the readme be updated to literally say this?

vadimstasiev avatar Feb 09 '23 21:02 vadimstasiev