-
Notifications
You must be signed in to change notification settings - Fork 886
Add Foundry Local as a chat and embedding provider in the AI Chat Web template #7586
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
base: main
Are you sure you want to change the base?
Changes from all commits
045c18b
d684624
33dba15
0434581
0256248
c977b7b
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -7,6 +7,8 @@ public class IngestedChunk | |
| { | ||
| #if (IsOllama) | ||
| public const int VectorDimensions = 384; // 384 is the default vector size for the all-minilm embedding model | ||
| #elif (IsFoundryLocal) | ||
| public const int VectorDimensions = 1024; // 1024 is the default vector size for the qwen3-embedding-0.6b embedding model | ||
| #else | ||
| public const int VectorDimensions = 1536; // 1536 is the default vector size for the OpenAI text-embedding-3-small model | ||
| #endif | ||
|
|
@@ -17,9 +19,15 @@ public class IngestedChunk | |
| #endif | ||
| public const string CollectionName = "data-AIChatWeb-CSharp.Web-chunks"; | ||
|
|
||
| #if (IsQdrant) | ||
| [VectorStoreKey(StorageName = "key")] | ||
| [JsonPropertyName("key")] | ||
| public required Guid Key { get; set; } | ||
| #else | ||
| [VectorStoreKey(StorageName = "key")] | ||
| [JsonPropertyName("key")] | ||
| public required string Key { get; set; } | ||
| #endif | ||
|
Comment on lines
+22
to
+30
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @adamsitnik Does this match your understanding that only Qdrant uses Guid keys?
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. My understanding is that Azure Foundry local provides an embedding generator (hence to proper VectorDimensions change above) but I don't think it provides any vector store. So we should not change the |
||
|
|
||
| [VectorStoreData(StorageName = "documentid")] | ||
| [JsonPropertyName("documentid")] | ||
|
|
||
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.
I think this is not needed, is also mentioned in the SDK reference docs: