Skip to content

.Net: Demo/process-framework-signalr#12374

Merged
markwallace-microsoft merged 17 commits into
microsoft:mainfrom
tommasodotNET:demo/process-framework-signalr
Aug 20, 2025
Merged

.Net: Demo/process-framework-signalr#12374
markwallace-microsoft merged 17 commits into
microsoft:mainfrom
tommasodotNET:demo/process-framework-signalr

Conversation

@tommasodotNET

Copy link
Copy Markdown
Contributor

Motivation and Context

This demo shows how to implement human-in-the-loop in Process Framework with SignalR. It is based on the ProcessWithCloudEvents one, readapting it to use SignalR.

Contribution Checklist

@tommasodotNET tommasodotNET requested a review from a team as a code owner June 4, 2025 19:01
@markwallace-microsoft markwallace-microsoft added .NET Issue or Pull requests regarding .NET code documentation labels Jun 4, 2025
@github-actions github-actions Bot changed the title Demo/process-framework-signalr .Net: Demo/process-framework-signalr Jun 4, 2025
@crickman crickman requested a review from alliscode June 5, 2025 00:19
@crickman crickman moved this to Sprint: In Review in Semantic Kernel Jun 5, 2025
@tommasodotNET

Copy link
Copy Markdown
Contributor Author

Adding reference to issue #10959 as this PR addresses that

@markwallace-microsoft

Copy link
Copy Markdown
Contributor

@tommasodotNET can you fix the merge conflict and we can get this merged

@tommasodotNET

Copy link
Copy Markdown
Contributor Author

@tommasodotNET can you fix the merge conflict and we can get this merged

done 😃

@markwallace-microsoft markwallace-microsoft added this pull request to the merge queue Aug 20, 2025
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Aug 20, 2025
@markwallace-microsoft

Copy link
Copy Markdown
Contributor

Merge failed due to unrelated integration test:

SemanticKernel.IntegrationTests.Connectors.OpenAI.OpenAIChatCompletionNonStreamingTests.TextGenerationShouldReturnMetadataAsync [FAIL]
[xUnit.net 00:04:04.13]       Microsoft.SemanticKernel.HttpOperationException : Service request failed.
[xUnit.net 00:04:04.13]       Status: 503 (Service Unavailable)
[xUnit.net 00:04:04.13]       
[xUnit.net 00:04:04.13]       ---- System.ClientModel.ClientResultException : Service request failed.
[xUnit.net 00:04:04.13]       Status: 503 (Service Unavailable)

@markwallace-microsoft markwallace-microsoft added this pull request to the merge queue Aug 20, 2025
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Aug 20, 2025
@markwallace-microsoft

Copy link
Copy Markdown
Contributor

Another unrelated integration test failure

Failed SemanticKernel.IntegrationTests.Connectors.OpenAI.OpenAIChatCompletionFunctionCallingTests.ConnectorAgnosticFunctionCallingModelClassesSupportSimulatedFunctionCallsAsync [31 s]
  Error Message:
   Polly.Timeout.TimeoutRejectedException : The operation didn't complete within the allowed timeout of '00:00:30'.
---- System.Threading.Tasks.TaskCanceledException : The operation was canceled.
-------- System.IO.IOException : Unable to read data from the transport connection: Operation canceled.
------------ System.Net.Sockets.SocketException : Operation canceled
  Stack Trace:
     at Polly.Utils.ExceptionUtilities.TrySetStackTrace[T](T exception)
   at Polly.Timeout.TimeoutResilienceStrategy.ExecuteCore[TResult,TState](Func`3 callback, ResilienceContext context, TState state)

@markwallace-microsoft markwallace-microsoft merged commit 55f94b5 into microsoft:main Aug 20, 2025
19 checks passed
@github-project-automation github-project-automation Bot moved this from Sprint: In Review to Sprint: Done in Semantic Kernel Aug 20, 2025
jcruzmot-te pushed a commit to thousandeyes/aia-semantic-kernel that referenced this pull request Sep 15, 2025
### Motivation and Context

This demo shows how to implement human-in-the-loop in Process Framework
with SignalR. It is based on the ProcessWithCloudEvents one, readapting
it to use SignalR.

### Contribution Checklist

<!-- Before submitting this PR, please make sure: -->

- [x] The code builds clean without any errors or warnings
- [x] The PR follows the [SK Contribution
Guidelines](https://github.com/microsoft/semantic-kernel/blob/main/CONTRIBUTING.md)
and the [pre-submission formatting
script](https://github.com/microsoft/semantic-kernel/blob/main/CONTRIBUTING.md#development-scripts)
raises no violations
- [x] All unit tests pass, and I have added new tests where possible
- [x] I didn't break anyone 😄

---------

Co-authored-by: Chris <66376200+crickman@users.noreply.github.com>
Co-authored-by: Chris Rickman <crickman@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation .NET Issue or Pull requests regarding .NET code

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants