-
Notifications
You must be signed in to change notification settings - Fork 3
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
Added SU rate for H100 GPU #86
base: main
Are you sure you want to change the base?
Conversation
"v100", | ||
"k80", | ||
"h100", | ||
]: # TODO Confirm this is the pci_name for H100s |
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.
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.
probably, but we don't know for sure.
@@ -41,6 +42,7 @@ class Rates(object): | |||
gpu_v100_su_name: str = "OpenStack GPUV100" | |||
gpu_a2_su_name: str = "OpenStack GPUA2" | |||
gpu_k80_su_name: str = "OpenStack GPUK80" | |||
gpu_h100_su_name: str = "OpenStack GPUH100" # TODO Confirm this is SU name for H100 |
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.
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.
That's what I'd call it. @joachimweyl does this name seem right?
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.
That sounds good to me.
Closes #85. The billing script should now be able to process H100 SUs. However, I have a few details I want to confirm before turning draft into a PR.