Skip to content
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

[UI] sky show-gpus only shows common gpus #1733

Open
ewzeng opened this issue Feb 27, 2023 · 2 comments
Open

[UI] sky show-gpus only shows common gpus #1733

ewzeng opened this issue Feb 27, 2023 · 2 comments
Labels
enhancement New feature or request P0

Comments

@ewzeng
Copy link
Collaborator

ewzeng commented Feb 27, 2023

sky show-gpus only shows some of the most common gpus, which can be confusing (e.g. here). Here are two potential solutions:

  1. Keep the current behavior, but in the output of sky show-gpus and sky show-gpus --cloud CLOUD, print
NOTE:  This list only contains the most common gpus. To see all gpus, please append `--all`.
  1. Add the above warning to sky show-gpus and make sky show-gpus --cloud CLOUD print out all the gpus of CLOUD.

What do people think?

@ewzeng ewzeng added the enhancement New feature or request label Feb 27, 2023
@concretevitamin
Copy link
Member

We should probably resolve this by 0.3.0 to for Lambda-only users.

Related:

» sky show-gpus --cloud lambda -a

shows

...
*NOTE*: for most GCP accelerators, INSTANCE_TYPE == (attachable) means the host VM's cost is not included.
...

which is surprising.

@ewzeng
Copy link
Collaborator Author

ewzeng commented Mar 1, 2023

We should probably resolve this by 0.3.0 to for Lambda-only users.

Related:

» sky show-gpus --cloud lambda -a

shows

...
*NOTE*: for most GCP accelerators, INSTANCE_TYPE == (attachable) means the host VM's cost is not included.
...

which is surprising.

Hmm, good idea!

@ewzeng ewzeng mentioned this issue Mar 9, 2023
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request P0
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants