-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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: Vector Store: Allowing json serializer options to be passed down from vector store options and user agent string. #8296
.Net: Vector Store: Allowing json serializer options to be passed down from vector store options and user agent string. #8296
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we add UTs to ensure this new functionality is working as expected?
@RogerBarreto Unfortunately the options passed to the azure ai search client is not exposed by the search client as a property, so you cannot verify afterwards that you passed in the right values. |
…n from vector store options and user agent string. (microsoft#8296) ### Motivation and Context See microsoft#8086 See microsoft#7580 To make it easier for users who are using the default construction for the azure ai search client to also use custom json serializer settings, adding this as an option to the vector store options class and using it when both when constructing the azure ai search client and the collection. Also adding the semantic kernel user agent string to the azure ai search client when constructing it. ### 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 😄
Motivation and Context
See #8086
See #7580
To make it easier for users who are using the default construction for the azure ai search client to also use custom json serializer settings, adding this as an option to the vector store options class and using it when both when constructing the azure ai search client and the collection.
Also adding the semantic kernel user agent string to the azure ai search client when constructing it.
Contribution Checklist