Skip to content

.Net Bug using OpenAI's ChatHistory AddStreamingMessageAsync extension #6153

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.

Sign up for GitHub

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

Jump to bottom
Closed
RogerBarreto opened this issue May 8, 2024 · 2 comments · Fixed by #9619
Closed

.Net Bug using OpenAI's ChatHistory AddStreamingMessageAsync extension #6153

RogerBarreto opened this issue May 8, 2024 · 2 comments · Fixed by #9619
Assignees
@RogerBarreto
Labels
bug Something isn't working .NET Issue or Pull requests regarding .NET code sk team issue A tag to denote issues that where created by the Semantic Kernel team (i.e., not the community)

Comments

@RogerBarreto
Copy link
Member

When calling AddStreamingMessageAsync with Auto Function Call with same chathistory in sequence, an error is thrown regarding the toolcall response not having an id not being available.

image
@RogerBarreto RogerBarreto added the bug Something isn't working label May 8, 2024
@RogerBarreto RogerBarreto self-assigned this May 8, 2024
@markwallace-microsoft markwallace-microsoft added .NET Issue or Pull requests regarding .NET code triage labels May 8, 2024
@RogerBarreto RogerBarreto added sk team issue A tag to denote issues that where created by the Semantic Kernel team (i.e., not the community) and removed triage labels May 8, 2024
@github-actions GitHub Actions
Copy link

github-actions bot commented Aug 7, 2024

This issue is stale because it has been open for 90 days with no activity.

@github-actions github-actions bot added the stale Issue is stale because it has been open for a while and has no activity label Aug 7, 2024
@github-actions GitHub Actions
Copy link

This issue was closed because it has been inactive for 14 days since being marked as stale.

@RogerBarreto RogerBarreto mentioned this issue Nov 8, 2024
Merged
@RogerBarreto RogerBarreto reopened this Nov 8, 2024
@RogerBarreto RogerBarreto removed triage stale Issue is stale because it has been open for a while and has no activity labels Nov 8, 2024
github-merge-queue bot pushed a commit that referenced this issue Nov 8, 2024
@RogerBarreto
.Net: Bugfix for AddMessageFromStreaming (#9619)
3a8899b 
### Motivation and Context

- Fixes #9458 
- Fixes #6153
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@RogerBarreto RogerBarreto

Labels
bug Something isn't working .NET Issue or Pull requests regarding .NET code sk team issue A tag to denote issues that where created by the Semantic Kernel team (i.e., not the community)
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

.Net: Bugfix for AddMessageFromStreaming RogerBarreto/semantic-kernel
2 participants
@RogerBarreto @markwallace-microsoft