-
Notifications
You must be signed in to change notification settings - Fork 620
Update sky show-gpus
#1752
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update sky show-gpus
#1752
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @ewzeng ! Looks like after we add tpu-v4 into our catalog, we need to update tpu list here
'tpu-v3-512', 'tpu-v3-1024', 'tpu-v3-2048' |
But the bigger issue is sky show-gpus
can be too long... I think we should consider hiding some tpu types or OTHER_GPU
by default and show hint msg. wdyt?
(sky-tmp) gcpuser@ray-dev-new-head-7cfd1b53-compute:~/tmp/skypilot$ sky show-gpus
COMMON_GPU AVAILABLE_QUANTITIES
V100 1, 2, 4, 8
V100-32GB 8
A100 1, 2, 4, 8, 16
A100-80GB 1, 2, 4, 8
P100 1, 2, 4
K80 1, 2, 4, 8, 16
T4 1, 2, 4, 8
M60 1, 2, 4
A10 1, 2
A10G 1, 4, 8
GOOGLE_TPU AVAILABLE_QUANTITIES
tpu-v2-8 1
tpu-v2-32 1
tpu-v2-128 1
tpu-v2-256 1
tpu-v2-512 1
tpu-v3-8 1
tpu-v3-32 1
tpu-v3-64 1
tpu-v3-128 1
tpu-v3-256 1
tpu-v3-512 1
tpu-v3-1024 1
tpu-v3-2048 1
OTHER_GPU AVAILABLE_QUANTITIES
A6000 1, 2, 4
Gaudi HL-205 8
K520 1, 4
P4 1, 2, 4
P40 1, 2, 4
RTX6000 1
Radeon MI25 1
Radeon Pro V520 1, 2, 4
T4g 1, 2
tpu-v4-1024 1
tpu-v4-1152 1
tpu-v4-128 1
tpu-v4-1280 1
tpu-v4-1408 1
tpu-v4-1536 1
tpu-v4-16 1
tpu-v4-1664 1
tpu-v4-1792 1
tpu-v4-1920 1
tpu-v4-2048 1
tpu-v4-2176 1
tpu-v4-2304 1
tpu-v4-2432 1
tpu-v4-256 1
tpu-v4-2560 1
tpu-v4-2688 1
tpu-v4-2816 1
tpu-v4-2944 1
tpu-v4-3072 1
tpu-v4-32 1
tpu-v4-3200 1
tpu-v4-3328 1
tpu-v4-3456 1
tpu-v4-3584 1
tpu-v4-3712 1
tpu-v4-384 1
tpu-v4-3840 1
tpu-v4-3968 1
tpu-v4-512 1
tpu-v4-64 1
tpu-v4-640 1
tpu-v4-768 1
tpu-v4-8 1
tpu-v4-896 1
'P100', | ||
'K80', | ||
'T4', | ||
'M60', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I thought our consensus in the meeting was to remove M60
and P100
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, yes. You are right.
It's a little bit counterintuitive to me that
|
Hmm, I agree that it's a bit counterintuitive. I kept it that way because that was the original behavior, but I am ok with changing it. |
Another issue is: do we support all global regions on AWS? as we are showing some Africa region |
I believe |
I see. I mean can we |
Oooo. I just tried, and got
so currently no. Does the T4 catalog include the region |
Should we get this into 0.3? @ewzeng @infwinston |
This PR is stale because it has been open 120 days with no activity. Remove stale label or comment or this will be closed in 10 days. |
This PR was closed because it has been stalled for 10 days with no activity. |
This pr updates
sky show-gpus
and fixes #1733.Changes:
sky show-gpus [--cloud CLOUD] [--region REGION]
shows all gpus satisfying the query, but does not show any detailed information (e.g. specs, pricing)--all
shows detailed informationCOMMON_GPU
(@WoosukKwon I forgot which ones you wanted to move out ofCOMMON_GPU
, could you remind me?)Tested:
sky show-gpus
sky show-gpus -a
sky show-gpus --cloud lambda
sky show-gpus --cloud lambda --region us-east-1
sky show-gpus --cloud lambda -a
sky show-gpus --cloud gcp
sky show-gpus --cloud gcp -a
sky show-gpus A100