-
Notifications
You must be signed in to change notification settings - Fork 5.4k
AITK Docs update - release 0.22.0 #8922
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?
Conversation
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.
@ms-johnalex Left some text suggestions. You might want to do a full pass for the use of "The" AI Toolkit - not sure what the official wording is.
 | ||
|
||
> [!TIP] | ||
> AI Toolkit can expose language models to enhance GitHub Copilot capabilities. To select an AI Toolkit model for GitHub Copilot, open the model picker and select **Manage Models**. Choose a model from the list of models under one of the following categories: |
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.
Is it AI Toolkit or "The" AI Toolkit? Noticed such update in other docs. There are several other instances still.
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 took out all the "the"s that I added.
> | ||
>  | ||
> | ||
> For more information, see [AI Toolkit models in GitHub Copilot](https://docs.github.com/en/copilot/using-github-copilot/ai-models/ai-toolkit-models-in-github-copilot). |
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.
Is this a new article that's still be released?
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'm not sure where that came from. Updated the link.
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.
The link is still not resolving. Should it be this: https://docs.github.com/en/copilot/how-tos/use-ai-models/change-the-chat-model
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.
yes, I updated again.
@ntrogh - I think these are ready for review! |
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.
@ms-johnalex I added some more feedback. There are still some suggestions open from the previous review, or is GH not showing your latest updates?
> | ||
>  | ||
> | ||
> For more information, see [AI Toolkit models in GitHub Copilot](https://docs.github.com/en/copilot/using-github-copilot/ai-models/ai-toolkit-models-in-github-copilot). |
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.
The link is still not resolving. Should it be this: https://docs.github.com/en/copilot/how-tos/use-ai-models/change-the-chat-model
Co-authored-by: Nick Trogh <[email protected]>
Co-authored-by: Nick Trogh <[email protected]>
Co-authored-by: Nick Trogh <[email protected]>
Co-authored-by: Nick Trogh <[email protected]>
Co-authored-by: Nick Trogh <[email protected]>
Co-authored-by: Nick Trogh <[email protected]>
Co-authored-by: Nick Trogh <[email protected]>
Co-authored-by: Nick Trogh <[email protected]>
…de-docs into upd-aitk-090925
Ok @ntrogh I think it's ready. |
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.
@ms-johnalex Two issues remaining.
docs/intelligentapps/overview.md
Outdated
| [Tracing](/docs/intelligentapps/tracing) | Monitor and analyze the performance of your AI applications. Collect and visualize trace data to gain insights into model behavior and performance. |  | | ||
|
||
## Who is AI Toolkit for? | ||
## Who is the AI Toolkit for? |
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.
One more "the" :)
@MuyangAmigo - Can you review when you get back? I think it's ready. |
No description provided.