Skip to content
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

.Net: Add sample to show how to create a decorator for generating embeddings. #9502

Merged

Conversation

westey-m
Copy link
Contributor

@westey-m westey-m commented Nov 1, 2024

Motivation and Context

Generating embeddings directly every time can be frustrating and it's nice to hide this functionality away from the main code

#9483

Description

Adding a sample that shows how to create a set of decorator classes for Vector Stores to automatically generate embeddings on upsert and when using VectorizableTextSearch.

Contribution Checklist

@westey-m westey-m requested a review from a team as a code owner November 1, 2024 17:52
@markwallace-microsoft markwallace-microsoft added the .NET Issue or Pull requests regarding .NET code label Nov 1, 2024
@github-actions github-actions bot changed the title Add sample to show how to create a decorator for generating embeddings. .Net: Add sample to show how to create a decorator for generating embeddings. Nov 1, 2024
Copy link
Member

@dmytrostruk dmytrostruk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great example! I like the approach with GenerateTextEmbedding attribute!

@westey-m westey-m added this pull request to the merge queue Nov 4, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Nov 4, 2024
@westey-m westey-m added this pull request to the merge queue Nov 4, 2024
Merged via the queue into microsoft:main with commit f3633e2 Nov 4, 2024
15 checks passed
@westey-m westey-m deleted the vector-store-textembeddingdecoratorsample branch November 4, 2024 13:57
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.

3 participants