Skip to content
This repository has been archived by the owner on Dec 4, 2023. It is now read-only.

issue-#202-resolved with nvidia-container install #9

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

ImayaBharathi
Copy link

resolved the following issue with nvidia-container toolkit instalation. added an echo is toolkit is not available and planning to install.

issue ref link

@filopedraz filopedraz requested a review from tiero June 29, 2023 15:57
@tiero
Copy link
Contributor

tiero commented Jul 1, 2023

Thanks! Are you using a public cloud provider or your own server with GPU to test?

@@ -54,6 +54,16 @@ PREM_AUTO_UPDATE=$PREM_AUTO_UPDATE" >$PREM_CONF_FOUND
curl --silent https://raw.githubusercontent.com/$USER/$REPO/main/docker-compose.gpu.yml -o ~/prem/docker-compose.gpu.yml
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please increase the script version to 0.0.2 at the top of the file

@ImayaBharathi
Copy link
Author

Thanks! Are you using a public cloud provider or your own server with GPU to test?

I am using a GPU machine from azure.

@ImayaBharathi
Copy link
Author

@tiero, increased the script version.

@tiero
Copy link
Contributor

tiero commented Jul 3, 2023

Doing some tests on AWS I noticed that we could also install the NVIDIA drivers automatically, beside the toolkit, but I ended up doing the same as you. Can you solve conflicts from the main to include latest changes, since I want to credit your contribution anyway ❤️

@ImayaBharathi
Copy link
Author

@tiero, apologies on the dealy, thanks for the words, i am not able resolve the conflict, it shows read only access, could you guide me on this, i would love to be a contributor!!!!

@tiero
Copy link
Contributor

tiero commented Oct 11, 2023

Can you try the latest script and tell me if this works for you?

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

Successfully merging this pull request may close these issues.

2 participants