MicrosoftDocs / semantic-kernel-docs

Semantic Kernel (SK) is a lightweight SDK enabling integration of AI Large Language Models (LLMs) with conventional programming languages.

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

The doc page "Chaining functions together" needs some cleanup

sramaswami11 opened this issue · comments

Please refer to the following url:
https://learn.microsoft.com/en-us/semantic-kernel/ai-orchestration/plugins/chaining-functions?tabs=Csharp
The code block right underneath the section "Using the RunAsync method to chain our functions" is confusing and redundant.
Redundant since the whole code block is repeated below under the section "Creating a semantic function that uses the new context variables" (and more complete there creating an saving off new context with the original _context).
Confusing since it has the line of code that tries to get the function "CreateResponse" which has not yet been created. Someone following this code will run into an error. So this whole block of code section needs to be removed. Or this code block can clarify that the CreateResponse function will be created later on this page.

I suspect this will be responded to more directly by posting in the semantic-kernel repo directly: https://github.com/microsoft/semantic-kernel)

I know this is under heavy review as part of the march towards release (currently beta3: https://github.com/microsoft/semantic-kernel/releases/tag/dotnet-1.0.0-beta3)