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: Adding a multi-store vector search sample. #8909

Open
wants to merge 4 commits into
base: feature-vector-search
Choose a base branch
from

Conversation

westey-m
Copy link
Contributor

Motivation and Context

One of the main features of the vector store abstractions is to be able to easily swap out one store for another, so showing an example where we are loading data into a store and doing vector searches with the same code, and only the top level registration differs for different vector stores.

Description

  • Adding a multi-store vector search example including Azure AI Search, Qdrant, Redis and Volatile
  • Renaming simple search example to match naming, and make small refactoring.

Contribution Checklist

@westey-m westey-m requested a review from a team as a code owner September 19, 2024 14:33
@markwallace-microsoft markwallace-microsoft added the .NET Issue or Pull requests regarding .NET code label Sep 19, 2024
@github-actions github-actions bot changed the title Adding a multi-store vector search sample. .Net: Adding a multi-store vector search sample. Sep 19, 2024

/// <summary>
/// An example showing how to use common code, that can work with any vector database, with an Azure AI Search instance.
/// The common code is in the <see cref="VectorStore_VectorSearch_MultiStore_Common"/> class.

Choose a reason for hiding this comment

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

The class name with _ looks odd to me. We use the _ in some of the classes that are tests for readability bit in this case it's a utility class.

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