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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

.Net: Adds Process Framework with Aspire demo #10614

Merged

Conversation

tommasodotNET
Copy link
Contributor

Motivation and Context

This demo aims to showcase how the Semantic Kernel Process Framework can be used with .NET Aspire. The Process Framework allows developers to create business processes based on events. Each step of the process can be an agent or native code.

Description

In this demo, I have defined the agents as external services, and each step will call these agents using HTTP requests. This setup allows .NET Aspire to add value by tracing the process using OpenTelemetry. Additionally, since each agent is a service, they can be restarted as needed using the .NET Aspire developer dashboard.

architecture

Why .NET Aspire?

Thanks to .NET Aspire, developers can leverage the developer dashboard to debug in real time the Process and all the agents running. Since the agents are single APIs, each can scale independently from each other.

aspire-dashboard
aspire-metrics
aspire-traces

Contribution Checklist

@tommasodotNET tommasodotNET requested a review from a team as a code owner February 19, 2025 17:16
@crickman crickman added .NET Issue or Pull requests regarding .NET code experimental Associated with an experimental feature enhancement samples agents labels Feb 19, 2025
@github-actions github-actions bot changed the title Adds process framework with aspire demo .Net: Adds process framework with aspire demo Feb 19, 2025
@crickman crickman changed the title .Net: Adds process framework with aspire demo .Net: Adds _Process Framework_ with _Aspire_ demo Feb 19, 2025
@crickman crickman changed the title .Net: Adds _Process Framework_ with _Aspire_ demo .Net: Adds Process Framework with Aspire demo Feb 19, 2025
tommasodotNET and others added 18 commits February 20, 2025 22:52
@tommasodotNET
Copy link
Contributor Author

tommasodotNET commented Feb 20, 2025

@crickman FYI I had to disable warning CA2234 when calling the translatoragent and summaryagent APIs because we are using customer HttpClient with baseaddress built via .net service discovery. Passing URIs instead of string would bypass the base address thus making the http call fail.
I documented this in the code.

Copy link
Contributor

@dluc dluc left a comment

Choose a reason for hiding this comment

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

I'd rename the endpoints to /api/summarize and /api/translate. Looks like some indentation issues creeped in, including Tabs 😱

@markwallace-microsoft markwallace-microsoft added this pull request to the merge queue Feb 21, 2025
Merged via the queue into microsoft:main with commit 4c91cfd Feb 21, 2025
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
agents enhancement experimental Associated with an experimental feature .NET Issue or Pull requests regarding .NET code samples
Projects
Status: Sprint: Done
Development

Successfully merging this pull request may close these issues.

4 participants