Skip to content

.Net: Fix samples and a small usability issue. - #12165

Merged
Adam Sitnik (adamsitnik) merged 2 commits into
microsoft:feature-vector-data-preb3from
westey-m:mevd-sample-fixes
May 19, 2025
Merged

Adam Sitnik (adamsitnik) merged 2 commits into
microsoft:feature-vector-data-preb3from
westey-m:mevd-sample-fixes

Conversation

@westey-m

Copy link
Copy Markdown
Contributor

Motivation and Context

Description

Contribution Checklist

@westey-m
westey (westey-m) requested a review from a team as a code owner May 19, 2025 13:27
@markwallace-microsoft Mark Wallace (markwallace-microsoft) added .NET Issue or Pull requests regarding .NET code kernel Issues or pull requests impacting the core kernel labels May 19, 2025
@github-actions github-actions Bot changed the title Fix samples and a small usability issue. .Net: Fix samples and a small usability issue. May 19, 2025

@adamsitnik Adam Sitnik (adamsitnik) left a comment

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.

LGTM!

@adamsitnik
Adam Sitnik (adamsitnik) merged commit 12342cc into microsoft:feature-vector-data-preb3 May 19, 2025
// Upsert the glossary entries into the collection.
await collection.UpsertAsync(glossaryEntries);

await Task.Delay(5000); // Add a wait to ensure that indexing completes before we continue.

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.

Ugh, this kind of thing always sucks :( But yeah, not sure what the alternative is for sample code...

This branch was previously deployed

1 inactive deployment
integration — 445f830d Deployed May 19, 2025 by westey-m
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 .NET Issue or Pull requests regarding .NET code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants