rsnk96 / Ubuntu-Setup-Scripts

Scripts to help you set up your Ubuntu quickly, especially if you're in any subfield of Data Science or AI!
Apache License 2.0
128 stars 76 forks source link

Added Workstation installs #14

Closed varun19299 closed 5 years ago

varun19299 commented 6 years ago

Added Nvidia-docker and docker.

Not adding DOCKERFILE, we are presently maintaining them at iitmcvg/workstation

rsnk96 commented 6 years ago

Is it possible to change the line where you wget Nvidia-Docker to the latest version?

For instance, this is how the latest version of Franz is retrieved

varun19299 commented 6 years ago

Sure , but Nvidia docker 1.0.1 supports most use cases (2 + requires a few changes)

On Mon 6 Aug, 2018, 5:05 PM R S Nikhil Krishna, notifications@github.com wrote:

Is it possible to change the line where you wget Nvidia-Docker to the latest version?

For instance, this https://github.com/rsnk96/Ubuntu-Setup-Scripts/blob/1b4e5c2dfd2d6c4486143aaf3ce2e07c4f09b1d0/2-GenSoftware.sh#L126 is how the latest version of Franz is retrieved

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/rsnk96/Ubuntu-Setup-Scripts/pull/14#issuecomment-410679015, or mute the thread https://github.com/notifications/unsubscribe-auth/AUME540iP0j_Y6s8oiPE2-brnCfRX8NRks5uOCoMgaJpZM4VoteB .

--

Thank you, Varun.

rsnk96 commented 6 years ago

It would be better for a general script if the latest version is used by default.

rsnk96 commented 5 years ago

The diff looks too complicated, and is not representative of the current diff (take .gitignore as per diff and in the current repo for instance)

Any chance you can convert this into a new PR with the latest version of nvidia-docker @varun19299 ?

rajat2004 commented 5 years ago

46 has been merged, this can be closed now