Skip to content

.Net: Avoid sending duplicate function tools when creating a thread - #10436

Merged
markwallace-microsoft merged 2 commits into
microsoft:mainfrom
markwallace-microsoft:users/markwallace/openaiassistant_functiontools_1
Feb 6, 2025
Merged

.Net: Avoid sending duplicate function tools when creating a thread#10436
markwallace-microsoft merged 2 commits into
microsoft:mainfrom
markwallace-microsoft:users/markwallace/openaiassistant_functiontools_1

Conversation

@markwallace-microsoft

Copy link
Copy Markdown
Contributor

Motivation and Context

Same issue that was fixed here #10413 but this time the fix is for OpenAI Assistants.

Description

Contribution Checklist

@markwallace-microsoft
markwallace-microsoft requested a review from a team as a code owner February 6, 2025 12:28
@markwallace-microsoft markwallace-microsoft added the .NET Issue or Pull requests regarding .NET code label Feb 6, 2025
@github-actions github-actions Bot changed the title Avoid sending duplicate function tools when creating a thread .Net: Avoid sending duplicate function tools when creating a thread Feb 6, 2025
Comment thread dotnet/src/Agents/OpenAI/OpenAIAssistantAgent.cs
@markwallace-microsoft
markwallace-microsoft added this pull request to the merge queue Feb 6, 2025
Merged via the queue into microsoft:main with commit 1b735c7 Feb 6, 2025
@markwallace-microsoft
markwallace-microsoft deleted the users/markwallace/openaiassistant_functiontools_1 branch February 6, 2025 17:54
jcruzmot-te pushed a commit to thousandeyes/aia-semantic-kernel that referenced this pull request Sep 15, 2025
…icrosoft#10436)

### Motivation and Context

Same issue that was fixed here
microsoft#10413 but this time
the fix is for OpenAI Assistants.

### Description

<!-- Describe your changes, the overall approach, the underlying design.
These notes will help understanding how your code works. Thanks! -->

### Contribution Checklist

<!-- Before submitting this PR, please make sure: -->

- [ ] The code builds clean without any errors or warnings
- [ ] The PR follows the [SK Contribution
Guidelines](https://github.com/microsoft/semantic-kernel/blob/main/CONTRIBUTING.md)
and the [pre-submission formatting
script](https://github.com/microsoft/semantic-kernel/blob/main/CONTRIBUTING.md#development-scripts)
raises no violations
- [ ] All unit tests pass, and I have added new tests where possible
- [ ] I didn't break anyone 😄
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

.NET Issue or Pull requests regarding .NET code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants