Skip to content

.Net MEVD: Test coverage for all kinds of data fields supported by Azure AI Search #12134

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

Open
wants to merge 6 commits into
base: feature-vector-data-preb3
Choose a base branch
from

Conversation

adamsitnik
Copy link
Member

No description provided.

@adamsitnik adamsitnik requested a review from roji May 16, 2025 16:50
@adamsitnik adamsitnik requested a review from a team as a code owner May 16, 2025 16:50
@markwallace-microsoft markwallace-microsoft added .NET Issue or Pull requests regarding .NET code kernel Issues or pull requests impacting the core kernel memory labels May 16, 2025
@@ -121,9 +121,10 @@ public static SearchFieldDataType GetSDKFieldDataType(Type propertyType)
Type t when t == typeof(bool) => SearchFieldDataType.Boolean,
Type t when t == typeof(int) => SearchFieldDataType.Int32,
Type t when t == typeof(long) => SearchFieldDataType.Int64,
// We don't map float to SearchFieldDataType.Single, because Azure AI Search doesn't support it.
// Half is also listed by the SDK, but currently not supported.
Copy link
Member Author

Choose a reason for hiding this comment

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

Source: I implemented the support and then got error from the Azure AI Search service

Embedding = ReadOnlyMemory<float>.Empty,
// From https://learn.microsoft.com/en-us/rest/api/searchservice/supported-data-types:
// "All of the above types are nullable, except for collections of primitive and complex types, for example, Collection(Edm.String)"
// So for collections, we can't use nulls.
Copy link
Member Author

Choose a reason for hiding this comment

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

This was unexpected ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kernel Issues or pull requests impacting the core kernel memory .NET Issue or Pull requests regarding .NET code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants