Incomplete checking in check_local_gpus

[WoosukKwon]

Hi,

Currently, the check_local_gpus function only checks whether the nvidia-smi is installed in the local machine.
However, this is incomplete as users may have installed nvidia-smi in the past (e.g., when using deep learning AMI on non-gpu instances).

Will make a quick PR to fix this.

Thanks

[concretevitamin]

Sounds great!

…

On Tue, Feb 22, 2022 at 11:41 Woosuk Kwon ***@***.***> wrote: Hi, Currently, the check_local_gpus function <https://github.com/sky-proj/sky/blob/6276b0fc76ee305470593aa7ee5d3a16fca898a2/sky/backends/backend_utils.py#L735> only checks whether the nvidia-smi is installed in the local machine. However, this is incomplete as users may have installed nvidia-smi in the past (e.g., when using deep learning AMI on non-gpu instances). Will make a quick PR to fix this. Thanks — Reply to this email directly, view it on GitHub <#410>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAEQWHX5UBLOY6IGLNFOIKDU4PRGHANCNFSM5PCK5WCA> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>. You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>