Skip to content

Conversation

@wadepickett
Copy link
Contributor

@wadepickett wadepickett commented Feb 27, 2025

Fixes #34636

Pulls in original commits and initial review commits from PR #33448.

  • Reworking this topic a bit as a sample guide for now and addressing a few more items that come up in the review.
  • Leaving the app sample to live in its own Microsoft repo where it is now after discussion with mikekistler Added article: SignalR with Open AI #33448 (comment) since the SignalR samples repo has been recently archived and is no longer an option.

Using sample docs from the dotnet/docs as a guide for this one which is not a full tutorial, but is a great sample guide (AKA sample readme)


Internal previews

📄 File 🔗 Preview link
aspnetcore/toc.yml aspnetcore/toc
aspnetcore/tutorials/ai-powered-group-chat/ai-powered-group-chat.md aspnetcore/tutorials/ai-powered-group-chat/ai-powered-group-chat

kevinguo-ed and others added 13 commits August 27, 2024 16:30
…at.md


Will be pulling all commits into a new PR.  Commiting suggestions here before that transfer starts.

Co-authored-by: Luke Latham <[email protected]>
@wadepickett wadepickett self-assigned this Feb 27, 2025
@wadepickett wadepickett changed the title Wadepickett/34636 signal rwith open ai SignalR with Open AI: Finish as Sample Guide Feb 27, 2025
@wadepickett wadepickett marked this pull request as ready for review March 20, 2025 23:01
@wadepickett wadepickett requested a review from IEvangelist March 20, 2025 23:01
@wadepickett
Copy link
Contributor Author

wadepickett commented Mar 20, 2025

Sorry this took so long to get to this. I had higher priority and security related work sitting before this one. It is an awesome example.

@kevinguo-ed, I made some changes based on all review suggestions. See if it looks OK to you.

@kevinguo-ed
Copy link
Contributor

@wadepickett Looks good. Thank you for the review.

@wadepickett wadepickett requested a review from IEvangelist March 21, 2025 19:18

The following diagram highlights the step-by-step communication and processing involved in using OpenAI services, employing an iterative approach to responses and data handling:

:::image type="content" source="./sequence-diagram-ai-powered-group-chat.png" alt-text="sequence diagram for the AI-powered group chat":::
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just an idea, not something that you need to do, but since we still do not have native support from the Learn platform for mermaid diagrams-I've been using mermaid.live and its share feature, that lets you encode/share diagrams. You can build this out, create the share, and embed the share as a comment in the Markdown (which hides it from the end user) but the content developer can use it as a source of truth for future edits, re-exporting it as an image to update the image itself.

For example:

<!--

https://mermaid.live/edit#pako:eNp1kk1P4zAQhv_KaK6botLQFnxAyhbt7mFRUQIXlMsQD6lFYmf9wVfV_752UyQQ4NPY7-N3xjPeYmMko0DH_wLrhi8UtZb6WkNcFLzRob9jO-4Hsl41aiDtoQJyUDUblqH7Si-S_lv5P-EOisYroz8zZWKuybYcYx7MZ2KdiLWNaZy35I2FX0F_47ZKbPEaLMPKuN44uPhZH7gKJufnP6AQcG1V27J143kxnpcCVhtuHhyY4EelM2aAS7IP0jwdTPYXJpPnUlQNabiP1dyUfw9OrOUHy7WAK3KO3TtmnaRJqqJkH6x2wI_UBUrvAcsudP5jXatoEmtVzjuQ5GkUJVNswSN5hgIz7Nn2pGQc4jbpNfoN91yjiKGMD6ix1rvIpWlWL7pB4W3gDK0J7QbFPXUu7sIQE7yN_w1hqWLPL8cvsv8pGcZeo9jiM4rl8uh0tjidneXzfDmbH88zfEGRnxxN48rzs3yaLxYn08Uuw1djounx_vbtPk4Jdv8Bo5TJ1A

-->

Check out this link.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mermaid: That is really nice, I love it.

@IEvangelist
Copy link
Member

Awesome! A few nits and FYIs, otherwise :shipit:

…at.md


commiting suggestion by IEvangelist

Co-authored-by: David Pine <[email protected]>
@wadepickett
Copy link
Contributor Author

@kevinguo-ed I am leaving your name on the author meta data: author: kevinguo-ed
However, I am changing ms.author to my name so that I also get notified for customer feedback and change requests, and work tracking for keeping this topic up to date.

@wadepickett wadepickett merged commit ae90dec into main Mar 21, 2025
3 checks passed
@wadepickett wadepickett deleted the wadepickett/34636SignalRwithOpenAI branch March 21, 2025 20:34
wadepickett added a commit that referenced this pull request Mar 23, 2025
* SignalR with Open AI: Finish as Sample Guide (#34848)

* Added article: SignalR with Open AI

* Added missing images and fixed formatting issues

* Added more details about managing chat history

* Update aspnetcore/tutorials/ai-powered-group-chat/ai-powered-group-chat.md

Will be pulling all commits into a new PR.  Committing suggestions here before that transfer starts.

Co-authored-by: Luke Latham <[email protected]>

* Update aspnetcore/tutorials/ai-powered-group-chat/ai-powered-group-chat.md

Co-authored-by: David Pine <[email protected]>

* Update aspnetcore/tutorials/ai-powered-group-chat/ai-powered-group-chat.md

Co-authored-by: David Pine <[email protected]>

* Update aspnetcore/tutorials/ai-powered-group-chat/ai-powered-group-chat.md

Co-authored-by: David Pine <[email protected]>

* Update aspnetcore/tutorials/ai-powered-group-chat/ai-powered-group-chat.md

Co-authored-by: David Pine <[email protected]>

* Update aspnetcore/tutorials/ai-powered-group-chat/ai-powered-group-chat.md

Co-authored-by: David Pine <[email protected]>

* Update aspnetcore/tutorials/ai-powered-group-chat/ai-powered-group-chat.md

Co-authored-by: David Pine <[email protected]>

* Update aspnetcore/tutorials/ai-powered-group-chat/ai-powered-group-chat.md

Co-authored-by: David Pine <[email protected]>

* SignalR with Open AI: pulling to new PR

* Change style to sample guide

* Edit pass

* Added instruction on code snippet

* Update toc.yml: samples repo and AI sample under Samples

* Text highlight

* Removed code link

* Added description for data flow diagram

* Small edit on diagram intro

* Update aspnetcore/tutorials/ai-powered-group-chat/ai-powered-group-chat.md

Co-authored-by: David Pine <[email protected]>

* Update aspnetcore/tutorials/ai-powered-group-chat/ai-powered-group-chat.md

committing suggestion by IEvangelist

Co-authored-by: David Pine <[email protected]>

* Update aspnetcore/tutorials/ai-powered-group-chat/ai-powered-group-chat.md

---------

Co-authored-by: Kevin Guo <[email protected]>
Co-authored-by: Luke Latham <[email protected]>
Co-authored-by: David Pine <[email protected]>

* add kubernetes-ingress.md to TOC (#34996)

* add kubernetes-ingress.md to TOC

* Add YARP Kubernetes Ingress Controller to TOC

* Fix link to Kubernetes Ingress sample

* Fix invalid XML in an example (#35028)

---------

Co-authored-by: Wade Pickett <[email protected]>
Co-authored-by: Kevin Guo <[email protected]>
Co-authored-by: Luke Latham <[email protected]>
Co-authored-by: David Pine <[email protected]>
Co-authored-by: Rick Anderson <[email protected]>
Co-authored-by: Jan Jones <[email protected]>
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.

Finish PR: SignalR with Open AI

4 participants