Skip to content

.Net: VectorStore: Remove experimental flag from connector implementations #8539

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.

Sign up for GitHub

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

Jump to bottom
Closed
westey-m opened this issue Sep 5, 2024 · 0 comments
Closed

.Net: VectorStore: Remove experimental flag from connector implementations #8539

westey-m opened this issue Sep 5, 2024 · 0 comments
Assignees
@westey-m
Labels
Ignite Features planned for next Ignite conference 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

Remove experimental flag from the vector store abstractions and implementations.
@westey-m westey-m added .NET Issue or Pull requests regarding .NET code memory connector Ignite Features planned for next Ignite conference 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
@westey-m westey-m moved this to Sprint: Planned in Semantic Kernel Sep 9, 2024
@westey-m westey-m changed the title .Net: VectorStore: Remove experimental flag from abstractions and implementations .Net: VectorStore: Remove experimental flag from connector implementations Nov 4, 2024
@westey-m westey-m mentioned this issue Nov 8, 2024
Merged
4 tasks
github-merge-queue bot pushed a commit that referenced this issue Nov 11, 2024
@westey-m
.Net: Remove experimental flag from VectorStore implementations. (#9624)
14bc62a 
### Motivation and Context

As part of graduating vector stores we want to remove the experimental
flag from the implementations and only keep them on memory store
implementations.

#8539

### Description

- Remove experimental flag from packages with vector store
implementations.
- Add experimental flag to any memory store files in the same package.
- Clean up warning suppressions in vector store projects.

### 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 😄
@westey-m westey-m closed this as completed Dec 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@westey-m westey-m

Labels
Ignite Features planned for next Ignite conference 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
Semantic Kernel
Archived in project
Milestone
No milestone
Development

No branches or pull requests
3 participants
@evchaki @markwallace-microsoft @westey-m