Skip to content

CompletenessEvaluator: Remove Seed parameter from ChatOptions #6519

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

Closed
wants to merge 1 commit into from

Conversation

planetis-m
Copy link

@planetis-m planetis-m commented Jun 16, 2025

Azure AI does not accept that parameter, which causes this evaluator to error out.

Diagnostics
Severity
Message
Error
Azure.RequestFailedException: {"detail":[{"type":"extra_forbidden","loc":["body","seed"],"msg":"Extra inputs are not permitted","input":123,"url":"https://errors.pydantic.dev/2.10/v/extra_forbidden"}]}
Status: 422 (Unprocessable Entity)
ErrorCode: Invalid input

Content:
{"error":{"code":"Invalid input","message":"{\"detail\":[{\"type\":\"extra_forbidden\",\"loc\":[\"body\",\"seed\"],\"msg\":\"Extra inputs are not permitted\",\"input\":123,\"url\":\"https://errors.pydantic.dev/2.10/v/extra_forbidden\"}]}","status":422}}
Microsoft Reviewers: Open in CodeFlow

Azure AI does not accept that parameter, which causes this evaluator to error out.
@planetis-m planetis-m requested a review from a team as a code owner June 16, 2025 07:37
@github-actions github-actions bot added the area-ai-eval Microsoft.Extensions.AI.Evaluation and related label Jun 16, 2025
@planetis-m planetis-m changed the title Remove Seed parameter from ChatOptions CompletenessEvaluator: Remove Seed parameter from ChatOptions Jun 16, 2025
@peterwald
Copy link
Member

@planetis-m Can you say which model version you are using that doesn't support seed?

@planetis-m
Copy link
Author

I can't reproduce anymore with gpt4.1. Must have been one of the low rate limit tier LLMs I used for testing. Can't remember really.

@planetis-m planetis-m closed this Jun 16, 2025
@shyamnamboodiripad
Copy link
Contributor

shyamnamboodiripad commented Jun 16, 2025

@planetis-m we generally want to keep the evaluators in this repo aligned with the ones in the Azure Python Evaluation SDK. The seed setting used here matches the one that is used in the corresponding python evaluator.

I would be surprised if this doesn't work for the models that are recommended for use with these evaluators (such as gpt-4o and newer). Please feel free to ping back / log an issue if you happen to run into this again. Thanks!

--

Also, it was my understanding that ChatOptions should generally be portable across models / model providers and that model / provider specific IChatClients would simply ignore options (like Seed) that they don't support (yet). However, specific clients may have reasons to diverge on this. Tagging @stephentoub and @eiriktsarpalis in case this is an incorrect assumption / in case you have any additional guidance around this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-ai-eval Microsoft.Extensions.AI.Evaluation and related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants