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

Official documentation missmatch with last README.md change #1474

Closed
MaKaNu opened this issue Mar 17, 2021 · 3 comments
Closed

Official documentation missmatch with last README.md change #1474

MaKaNu opened this issue Mar 17, 2021 · 3 comments

Comments

@MaKaNu
Copy link

MaKaNu commented Mar 17, 2021


1. Issue description

The official documentation describes to use nvidia-docker2 but before commit 88a2fda the README shows to use nvidia-container-toolkit because nvidia-docker2 is deprecated.

2. Steps to reproduce the issue

Compare official Documentation Chapter "Setting up NVIDIA Container Toolkit" with diff of commit 88a2fda line 32

@klueska
Copy link
Contributor

klueska commented Mar 17, 2021

The nvidia-docker2 package is very much not deprecated. It is unfortunate that it was ever announced as such.

You should use whatever is appropriate for your use case (and more often than not this is nvidia-docker2 not simply nvidia-container-toolkit). Please see this comment I made a while back for a longer explanation:

#1268 (comment)

@klueska klueska closed this as completed Mar 17, 2021
@MaKaNu
Copy link
Author

MaKaNu commented Mar 18, 2021

Thx klueska, I think there should be a short reference because of diverse stackoverflow questions the message of deprecation is spreading and confusing still a lot of people.

@navierb
Copy link

navierb commented May 18, 2021

It is very unfortunate. The documentation is a mess.

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

No branches or pull requests

3 participants