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: VectorStore - Add generic data model and mappers #8536

Open
westey-m opened this issue Sep 5, 2024 · 0 comments
Open

.Net: VectorStore - Add generic data model and mappers #8536

westey-m opened this issue Sep 5, 2024 · 0 comments
Assignees
Labels
Ignite memory connector .NET Issue or Pull requests regarding .NET code sk team issue A tag to denote issues that where created by the Semantic Kernel team (i.e., not the community)

Comments

@westey-m
Copy link
Contributor

westey-m commented Sep 5, 2024

Add a generic data model so that vector stores can be used without needing to define your own data type.
Add a mapper for each vector store that allows usage of the generic data model.

@westey-m westey-m added .NET Issue or Pull requests regarding .NET code memory connector Ignite labels Sep 5, 2024
@westey-m westey-m self-assigned this Sep 5, 2024
@westey-m westey-m removed the triage label Sep 5, 2024
@evchaki evchaki added the sk team issue A tag to denote issues that where created by the Semantic Kernel team (i.e., not the community) label Sep 5, 2024
github-merge-queue bot pushed a commit that referenced this issue Sep 11, 2024
…earch mapper. (#8666)

### Motivation and Context

For cases where a developer is defining a schema using configuration,
it's useful to have a generic data model that can be used with any data
source. A separate definition can be supplied in this case, that defines
the schema, and this can look different to the data model.

### Description

- Adding a generic data model for any vector store records
- Adding a mapping for Azure AI Search that uses the definition to map
from the database record to the generic data model.

#8536 

### Contribution Checklist

<!-- Before submitting this PR, please make sure: -->

- [x] The code builds clean without any errors or warnings
- [x] The PR follows the [SK Contribution
Guidelines](https://github.com/microsoft/semantic-kernel/blob/main/CONTRIBUTING.md)
and the [pre-submission formatting
script](https://github.com/microsoft/semantic-kernel/blob/main/CONTRIBUTING.md#development-scripts)
raises no violations
- [x] All unit tests pass, and I have added new tests where possible
- [x] I didn't break anyone 😄
github-merge-queue bot pushed a commit that referenced this issue Sep 11, 2024
…earch mapper. (#8666)

### Motivation and Context

For cases where a developer is defining a schema using configuration,
it's useful to have a generic data model that can be used with any data
source. A separate definition can be supplied in this case, that defines
the schema, and this can look different to the data model.

### Description

- Adding a generic data model for any vector store records
- Adding a mapping for Azure AI Search that uses the definition to map
from the database record to the generic data model.

#8536 

### Contribution Checklist

<!-- Before submitting this PR, please make sure: -->

- [x] The code builds clean without any errors or warnings
- [x] The PR follows the [SK Contribution
Guidelines](https://github.com/microsoft/semantic-kernel/blob/main/CONTRIBUTING.md)
and the [pre-submission formatting
script](https://github.com/microsoft/semantic-kernel/blob/main/CONTRIBUTING.md#development-scripts)
raises no violations
- [x] All unit tests pass, and I have added new tests where possible
- [x] I didn't break anyone 😄
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Ignite memory connector .NET Issue or Pull requests regarding .NET code sk team issue A tag to denote issues that where created by the Semantic Kernel team (i.e., not the community)
Projects
Status: Sprint: In Review
Development

No branches or pull requests

3 participants