Skip to content
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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Simplified and updated embeddings doc #40880

Closed

Conversation

jmatthiesen
Copy link
Contributor

@jmatthiesen jmatthiesen commented May 14, 2024

Made some changes to simplify the document and focus on the core details of what embeddings are, leaving other documents to describe the details of using them.

  • Re-ordered the content so it's:
    • Description of concept
    • Example use cases
    • Summary of how you'd use embeddings with links off to appropriate docs

Removed some links to Semantic Kernel content that was a bit confusing or not necessarily relevant (e.g. Prompt Template Syntax).


Internal previews

馃搫 File 馃敆 Preview link
docs/ai/conceptual/embeddings.md Embeddings in .NET

Made some changes to simplify the document and focus on the core details of what embeddings are, leaving other documents to describe the details of using them.
@dotnet-bot dotnet-bot added this to the May 2024 milestone May 14, 2024
@jmatthiesen jmatthiesen marked this pull request as ready for review May 17, 2024 14:58
@jmatthiesen jmatthiesen requested a review from a team as a code owner May 17, 2024 14:58
@gewarren gewarren changed the base branch from new-conceptual-ai to main May 21, 2024 22:01
@gewarren gewarren changed the base branch from main to new-conceptual-ai May 21, 2024 22:02
@gewarren
Copy link
Contributor

@jmatthiesen I tried to change the base branch to main, but that brough in 50 extra files. You probably need to create a new branch off main and resubmit this PR.

@alexwolfmsft
Copy link
Contributor

These changes went in with a different branch, I handled this for the initial release but forgot to close the PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants