-
Notifications
You must be signed in to change notification settings - Fork 5.8k
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
Fixes/updates for AI Conceptual docs #40848
Closed
jmatthiesen
wants to merge
181
commits into
new-conceptual-ai
from
u/jmatthiesen/aiConceptualDocFixes
Closed
Fixes/updates for AI Conceptual docs #40848
jmatthiesen
wants to merge
181
commits into
new-conceptual-ai
from
u/jmatthiesen/aiConceptualDocFixes
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Summary * Microsoft package repository: - stops shipping .NET staring with Ubuntu 24.04 (see dotnet/core#9258) * .NET backports package repository (https://launchpad.net/~dotnet/+archive/ubuntu/backports): - add support information - list supported versions - add (un)install instructions - add & change troubleshooting instructions * Update outdated ARM information * Update IBM System Z platform support infromation * Update Dependencies: - add ca-certificates (see https://bugs.launchpad.net/ubuntu/+source/dotnet6/+bug/2057982 & dotnet/sdk#39544) - change libicu to version 74 for Ubuntu 24.04+ (see https://launchpad.net/ubuntu/noble/+source/icu)
* Fix typo in cross-platform-cryptography.md I believe given the context, the sentence: > As **authentication** encryption requires ... Should read: > As **authenticated** encryption requires ... It may be worth going further and changing "As" to "Since" so it reads: > **Since authenticated** encryption requires ... ...but I defer to the PR reviewers on that extra tweak. * Apply suggestions from code review --------- Co-authored-by: David Pine <david.pine@microsoft.com>
Fix link for `editorconfig` template row
* update for Xamarin archive Replace /xamarin with /maui * Remove xamarin in prep for achival As Xamarin products are delisted, don't recommend them.
…ve-update.md Co-authored-by: Genevieve Warren <24882762+gewarren@users.noreply.github.com>
…ve-update.md Co-authored-by: Genevieve Warren <24882762+gewarren@users.noreply.github.com>
…ve-update.md Co-authored-by: Genevieve Warren <24882762+gewarren@users.noreply.github.com>
…ve-update.md Co-authored-by: Genevieve Warren <24882762+gewarren@users.noreply.github.com>
…ve-update.md Co-authored-by: Genevieve Warren <24882762+gewarren@users.noreply.github.com>
…ve-update.md Co-authored-by: Genevieve Warren <24882762+gewarren@users.noreply.github.com>
Co-authored-by: Jan Kotas <jkotas@microsoft.com>
The description of variable capture was a bit confusing; the local function won't be emitted as a delegate type here. However, it will use a closure, just like delegate types do, so I've updated the wording to reflect that.
fixed minor syntax for yield snippet
* Convert DllImport uses to LibraryImport There are several places where DllImport remains. This is due to the nature of some of the documentation and how it is intended to be consumed. * Update to .NET 8 TFM. --------- Co-authored-by: Genevieve Warren <24882762+gewarren@users.noreply.github.com> Co-authored-by: Rich Lander <rlander@microsoft.com> Co-authored-by: Elinor Fung <elfung@microsoft.com>
* Rewrite samples; update links & minor text * add prereqs; move file note * md
* Update import-declarations-the-open-keyword.md Chapter on global keyword * better names * specifier not keyword * Update import-declarations-the-open-keyword.md Co-authored-by: Petr <psfinaki@users.noreply.github.com> * Update import-declarations-the-open-keyword.md Co-authored-by: Petr <psfinaki@users.noreply.github.com> * try fix md linting errors --------- Co-authored-by: Petr <psfinaki@users.noreply.github.com>
jmatthiesen
commented
May 10, 2024
@@ -33,8 +33,6 @@ This text could be tokenized as: | |||
- a | |||
- cat | |||
|
|||
The model will add only the first occurrence of the "a" token to its vocabulary. All other occurrences can be represented by that first token. |
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.
Removed this line to simplify the section since it wasn't core to it and it is also described later in the doc with the example given for adding IDs to tokens.
* add example for order of execution * used wrong reference * not sure for what purpose this identifier is used but adjust it so it fits the format of the other examples
elif directive without a condition
* AI - chat refresh * Update docs/ai/get-started-app-chat-template.md Co-authored-by: Genevieve Warren <24882762+gewarren@users.noreply.github.com> * Update docs/ai/get-started-app-chat-template.md Co-authored-by: Genevieve Warren <24882762+gewarren@users.noreply.github.com> --------- Co-authored-by: Genevieve Warren <24882762+gewarren@users.noreply.github.com>
* Add note about collection expressions. Fixes #38712 Note that these examples all use collection expressions. * Update docs/csharp/language-reference/builtin-types/arrays.md Co-authored-by: Genevieve Warren <24882762+gewarren@users.noreply.github.com> * show both examples in note. --------- Co-authored-by: Genevieve Warren <24882762+gewarren@users.noreply.github.com>
OpenStreamAsync => OpenOutboundStreamAsync AcceptStreamAsync => AcceptInboundStreamAsync
…ptualDocFixes to stay in sync
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
WORK IN-PROGRESS:
Collecting updates to the new conceptual AI docs as part of a team "docs bash".
Fixes included:
Internal previews
Toggle expand/collapse