[JarvisLabs] Support RTX PRO 6000; update API endpoint; remove spot offers#235
Conversation
5e97532 to
f5fd15c
Compare
f5fd15c to
5e97532
Compare
5e97532 to
e5dd10d
Compare
| if gpu_name == gpu_type and any(c.isspace() for c in gpu_type): | ||
| return None |
There was a problem hiding this comment.
(nit) Now that you pass gpu_type in provider_data, it should be possible to normalize GPU names with spaces instead of skipping them (like you suggested in the initial JarvisLabs PR versiom)
There was a problem hiding this comment.
I’d prefer to keep this fail-closed. provider_data["gpu_type"] solves the create-time token issue once we have a reviewed mapping, but it does not make arbitrary spaced GPU names safe to canonicalize. For example, blindly stripping spaces would map RTX A6000 to RTXA6000, while gpuhunt uses A6000.
So I kept unmapped spaced JarvisLabs GPU types skipped with a warning and renamed the warning from “ambiguous” to “unmapped”. For any spaced JarvisLabs token we verify, we can add an explicit mapping.
RTXPRO6000and store the original JarvisLabs GPU type inprovider_datawhen needed.backendn.jarvislabs.netfor JarvisLabs offer discovery. JarvisLabs confirmedbackendprod.jarvislabs.netandbackendn.jarvislabs.netnow point to the same region and function identically.