Skip to content

Conversation

@SBrandeis
Copy link
Contributor

@SBrandeis SBrandeis commented Jul 7, 2025

Document how we validate inference provider mappings

Sorry for all the whitespace changes 😬 My linter

@HuggingFaceDocBuilderDev

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

Copy link
Member

@Vaibhavs10 Vaibhavs10 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The validation part looks good! thanks for adding it! 🤗

@julien-c
Copy link
Member

julien-c commented Jul 7, 2025

care pushing just the part that changed? (sorry)

@julien-c
Copy link
Member

julien-c commented Jul 7, 2025

git add > partial commit

Comment on lines +327 to +325
For large language models (LLMs), additional behavioral tests are conducted:

- Tool calling support.
- Structured output support.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

does this mark the model as in error as well? or it's a ≠ UI? (seems a bit heavy to mark it as Failing validation for this)

Copy link
Contributor Author

@SBrandeis SBrandeis Jul 7, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It does not - for now it just feeds a map of booleans flagging wether or not the provider-llm pair supports the feature

The tests are only carried out if the initial test succeeds (first token received in < 5s)

It's not displayed nor used anywhere yet

@SBrandeis SBrandeis force-pushed the inference-providers-validation branch from 8225a6e to 91dcdf7 Compare July 7, 2025 16:36
@SBrandeis SBrandeis requested review from Vaibhavs10 and julien-c July 7, 2025 16:38
@SBrandeis SBrandeis merged commit 4fe884b into main Jul 8, 2025
2 checks passed
@SBrandeis SBrandeis deleted the inference-providers-validation branch July 8, 2025 10:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants