-
-
Notifications
You must be signed in to change notification settings - Fork 530
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
Add implementations of Microsoft.Extensions.AI's IChatClient / IEmbeddingGenerator #659
Conversation
…dingGenerator This enables OpenAiService to then be consumed by anyone in the .NET ecosystem written in terms of these exchange types.
Hmm, that's interesting. Not sure how I missed this announcement. I will check this after the real-time API implementation completed. |
Reorganized `using` directives and added new namespaces. Modified `ChatClientMetadata` for better readability. Expanded `GetService` and `Dispose` methods to use block bodies. Refactored `CompleteAsync`, `CompleteStreamingAsync`, `CreateRequest`, and other methods to use `var` and simplify loops and statements. Improved error messages and renamed variables for clarity.
@stephentoub Is there any documentation for changelogs? I'm struggling to find details about the changes and how to address the breaking changes. |
I thought I submitted a pr to update this library, but it looks like I never hit submit. I can do so when I'm back at my computer. Regarding a changelog, yes: I expect the only break to address is that GetService became non-generic. |
That would be great, Thanks! |
This enables OpenAiService to then be consumed by anyone in the .NET ecosystem written in terms of these exchange types.
Examples in https://www.nuget.org/packages/Microsoft.Extensions.AI.Abstractions/9.0.0-preview.9.24525.1#readme-body-tab.
More details in https://devblogs.microsoft.com/dotnet/introducing-microsoft-extensions-ai-preview/.