Skip to content

.Net: Update ONNX Rag sample to use vector store. - #9546

Merged
westey (westey-m) merged 2 commits into
microsoft:mainfrom
westey-m:vector-store-update-onnx-rag
Nov 5, 2024
Merged

westey (westey-m) merged 2 commits into
microsoft:mainfrom
westey-m:vector-store-update-onnx-rag

Conversation

@westey-m

Copy link
Copy Markdown
Contributor

Motivation and Context

#9377

Description

Update ONNX Rag sample to use vector store.

Contribution Checklist

@westey-m
westey (westey-m) requested a review from a team as a code owner November 5, 2024 15:24
@markwallace-microsoft Mark Wallace (markwallace-microsoft) added the .NET Issue or Pull requests regarding .NET code label Nov 5, 2024
@github-actions github-actions Bot changed the title Update ONNX Rag sample to use vector store. .Net: Update ONNX Rag sample to use vector store. Nov 5, 2024
@rogerbarreto

Copy link
Copy Markdown
Member

This is an excellent use case for Migration Guidance. Will save this PR as an example BTW :)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Would keep it simple (without Handler bars) or add this as an option.

@westey-m
westey (westey-m) added this pull request to the merge queue Nov 5, 2024
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Nov 5, 2024
@westey-m
westey (westey-m) added this pull request to the merge queue Nov 5, 2024
Merged via the queue into microsoft:main with commit 8f81caf Nov 5, 2024
@westey-m
westey (westey-m) deleted the vector-store-update-onnx-rag branch November 5, 2024 18:00

This branch was previously deployed

1 inactive deployment
integration — e09688ac Deployed Nov 5, 2024 by westey-m
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