-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
.Net Agents - Support role-override for ChatCompletionAgent
#10601
Conversation
ChatCompletionAgent
ChatCompletionAgent
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.
Copilot reviewed 1 out of 1 changed files in this pull request and generated no comments.
Comments suppressed due to low confidence (1)
dotnet/src/Agents/Core/ChatCompletionAgent.cs:59
- [nitpick] The new property 'InstructionsRole' defaults to AuthorRole.System; consider enhancing the XML documentation to clearly indicate that when targeting deep-reasoning models (which require 'developer'), users need to override this property accordingly.
public AuthorRole InstructionsRole { get; init; } = AuthorRole.System;
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.
Copilot reviewed 1 out of 1 changed files in this pull request and generated no comments.
Motivation and Context
The "O*" series deep-reasoning models from OpenAI do not support
System
role. For certain versions, support was added for aDeveloper
role.Description
Allow
ChatCompletionAgent
make use of "O*" series models by supportingInstructionsRole
that may be optionally overriden.Contribution Checklist