-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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: Kernel.RunAsync() remove legacy call patterns (including params) #3143
Comments
Per #3145
|
FYI https://github.com/microsoft/semantic-kernel/blob/main/dotnet/README.md the basic sample doesn't work anymore because the nice clear RunAsync(string) now wants a Microsoft.SemanticKernel.Orchestration.SKContext. Not sure why SK is needed as a type prefix when the large namespace handles scoping for you, or why this internal type is exposed at all. During this refactor, what is the clean "hello world" for .NET? This was clear (like Python's is). I'd encourage us to not forget the '5 lines to wow' crowd as you refactor for the enterprise. (this no longer works, as you know)
|
@shanselman, yup, the "keep simple things easy" is key. With what's now in main, that's this: var summarize = kernel.CreateFunctionFromPrompt(prompt);
Console.WriteLine(await summarize.InvokeAsync(kernel, "how are you")); There's also a Console.WriteLine(await kernel.InvokePromptAsync("What is the color of the sky?")); |
We have these new "Getting Started" samples which show simplex things are easy and advanced scenarios are supported with very little extra code https://github.com/microsoft/semantic-kernel/blob/main/dotnet/samples/KernelSyntaxExamples/Getting_Started/Step1_Create_Kernel.cs |
Important
Labeled Urgent because it will require a breaking change.
The text was updated successfully, but these errors were encountered: