SendAudioAsync does not set 'isSendingAudio' back to false #268
Comments
|
How is this still not fixed? Isn't there a failing test suite somewhere? |
|
Bump. Although the code appears refactored, either this is still an issue or the dependent microsoft library hasn't been updated for it to work. |
|
Thank you for your contribution, @onionhammer ! Have you tried the latest stable 2.1.0 version from early December? I believe this issue is fixed in that one, but let me know if you're still seeing it. Here's a link: |
|
Looks like the issue is that the only (working) version of 'Azure.AI.OpenAI' is referencing the broken version of your nuget package. |
|
Ah, so this is an issue with the Azure OpenAI library (Azure.AI.OpenAI) rather than the OpenAI library? Got it. I think I see what the problem is now: Azure OpenAI versions a little differently than OpenAI. In particular, a stable version of the Azure OpenAI service does not include beta APIs (such as the Realtime API). Since version 2.1.0 of the Azure OpenAI library is a stable version and targets the latest stable version of the Azure service, it does not include beta APIs either, and that is why the fix is missing there. We're preparing to publish new 2.2.0-beta.1 versions of both libraries in a few days. This beta library of Azure OpenAI will target the beta versions of the service, and therefore it will once again include the Realtime APIs (along with this fix). I will post back here as soon as it is out. |
|
Ah that makes sense, thanks! |
Service
OpenAI
Describe the bug
openai-dotnet/src/Custom/RealtimeConversation/RealtimeConversationSession.cs
Line 95 in b8ec620
In the 'BinaryData' overload, subsequent calls will fail with exception since
_isSendingAudiois not set back tofalseas it does in the memorystream overloadSteps to reproduce
This is appears to be a logical/code error
Code snippets
No response
OS
.NET version
.NET 8
Library version
2.1.0-*
The text was updated successfully, but these errors were encountered: