Repository navigation
Fix dynamic SUT listing for OpenAI and Together dedicated - #1650
kaulastudies wants to merge 1 commit into
Conversation
|
MLCommons CLA bot All contributors have signed the MLCommons CLA ✍️ ✅ |
|
recheck |
77fe4fe to
57fdaf5
Compare
|
Refreshed this PR against current Validation on the refreshed branch:
I also ran the full suite locally on Windows. Its remaining failures are environment-specific and unrelated to this diff (Linux-only Together serverless behavior remains unchanged. Ready for renewed review when convenient. |
|
@mlcommons/ai-safety-engineers Could a maintainer review the current OpenAI/Together dedicated discovery patch, The latest build and CLA checks pass. The scope remains limited to providers that can enumerate their models or dedicated endpoints; Together serverless behavior is unchanged. The refreshed regression and type-check results are recorded above. If the preferred API or project direction has changed, I can adjust the patch. Thank you. |
Summary
Fixes dynamic SUT listing for two provider paths that can authoritatively enumerate models/endpoints:
maker/modeland bare deployment names to SUT definitions.This intentionally does not change:
Duplicate Together dedicated deployments are de-duplicated by SUT UID, and malformed/empty deployment entries are ignored.
Validation
tests/modelgauge_tests/test_dynamic_sut_factory.py: passed in isolationtests/modelgauge_tests/test_sut_factory.py: passed in a fresh pytest processgit diff --check: passedThe two regression modules are run in separate pytest processes because
test_dynamic_sut_factory.pyintentionally defines invalidDynamicDriverSUTFactorysubclasses while testingDRIVER_NAMEvalidation; runningtest_sut_factory.pyafterward in the same interpreter can expose those temporary subclasses to factory auto-discovery.All new provider-listing behavior is covered with mocked, network-free tests.
Closes #1644