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.

Sign up for GitHub

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

Jump to bottom

Python: .Net: Processes - Cloud Events supporting components for emitting events + MicrosoftGraph emit Demo #9712

Draft
wants to merge 26 commits into
base: main
Choose a base branch
from
Draft

Python: .Net: Processes - Cloud Events supporting components for emitting events + MicrosoftGraph emit Demo #9712

wants to merge 26 commits into from

Conversation

esttenorio
Copy link
Contributor

@esttenorio esttenorio commented Nov 14, 2024 •
edited
Loading

Description

Adding components to allow "forwarding" internal events externally and allow a way to subscribe to them outside the SK Process.

Includes:

  • Piped EventName in ProcessStepEdgeBuilder
  • Piped SourceEventName and EventId into KernelProcessEdge and ProcessMessage
  • Adding "EmitMessageExternally" in LocalProcess
  • New components KernelProcessEventsSubscriber and KernelProcessEventsSubscriberInfo(passed to KernelProcess)
  • Demo including 2 ways of emitting cloud events: using steps and using KernelProcessEventsSubscriber approach -> More details in dotnet/samples/Demos/PorcessWithCloudEvents/README.md

Fixes #9722
Fixes #9721

Demo

image

Contribution Checklist

@esttenorio esttenorio added experimental Associated with an experimental feature processes labels Nov 14, 2024
@esttenorio esttenorio self-assigned this Nov 14, 2024
@markwallace-microsoft markwallace-microsoft added the .NET Issue or Pull requests regarding .NET code label Nov 14, 2024
@github-actions github-actions bot changed the title .NET: Processes - Cloud Events supporting components + Demos of usabe .Net: Processes - Cloud Events supporting components + Demos of usabe Nov 14, 2024
@esttenorio esttenorio changed the title .Net: Processes - Cloud Events supporting components + Demos of usabe .Net: Processes - Cloud Events supporting components for emitting events + MicrosoftGraph emit Demo Nov 16, 2024
esttenorio
esttenorio commented Nov 16, 2024
View reviewed changes
dotnet/src/Experimental/Process.Abstractions/KernelProcessEventsSubscriberInfo.cs
method.Invoke(instance, [data]);
}
}
}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

this is only used in LocalProcess, perhaps need to create a LocalKernelProcessEventsSubscriber that focuses on execution only

@markwallace-microsoft markwallace-microsoft added the python Pull requests for the Python Semantic Kernel label Dec 11, 2024
@github-actions github-actions bot changed the title .Net: Processes - Cloud Events supporting components for emitting events + MicrosoftGraph emit Demo Python: .Net: Processes - Cloud Events supporting components for emitting events + MicrosoftGraph emit Demo Dec 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@alliscode alliscode Awaiting requested review from alliscode

@crickman crickman Awaiting requested review from crickman

At least 2 approving reviews are required to merge this pull request.

Assignees

@esttenorio esttenorio

Labels
documentation experimental Associated with an experimental feature .NET Issue or Pull requests regarding .NET code processes python Pull requests for the Python Semantic Kernel
Projects
Semantic Kernel
Status: Sprint: In Progress
Milestone
No milestone
2 participants
@esttenorio @markwallace-microsoft