Skip to content

.Net: Copy AzureOpenAITextEmbeddingGenerationService to Connectors.AzureOpenAI project - #7022

Merged
SergeyMenshykh merged 2 commits into
microsoft:feature-connectors-openaifrom
SergeyMenshykh:azure-text-embedding-service-preparation
Jul 1, 2024
Merged

.Net: Copy AzureOpenAITextEmbeddingGenerationService to Connectors.AzureOpenAI project#7022
SergeyMenshykh merged 2 commits into
microsoft:feature-connectors-openaifrom
SergeyMenshykh:azure-text-embedding-service-preparation

Conversation

@SergeyMenshykh

Copy link
Copy Markdown
Contributor

Motivation and Context

This PR prepares the AzureOpenAITextEmbeddingGenerationService for migration to the new Azure AI SDK v2. The AzureOpenAITextEmbeddingGenerationService is copied to the Connectors.AzureOpenAI project as is and excluded from compilation to simplify code review of the next PR, which will refactor it to use the new Azure SDK. The next PR will also add unit/integration tests, along with service collection and kernel builder extension methods.

Description

The AzureOpenAITextEmbeddingGenerationService class was copied as is to the Connectors.AzureOpenAI project. The class build action was set to none to exclude it temporarily from the compilation process until it gets refactored to the new SDK.

@SergeyMenshykh
SergeyMenshykh requested a review from a team as a code owner July 1, 2024 08:59
@SergeyMenshykh SergeyMenshykh self-assigned this Jul 1, 2024
@markwallace-microsoft Mark Wallace (markwallace-microsoft) added .NET Issue or Pull requests regarding .NET code kernel Issues or pull requests impacting the core kernel labels Jul 1, 2024
@SergeyMenshykh
SergeyMenshykh merged commit 05374c8 into microsoft:feature-connectors-openai Jul 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kernel Issues or pull requests impacting the core kernel .NET Issue or Pull requests regarding .NET code

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants