.Net: Function calling clean-up (#9521)

### Motivation, Context and Documentation Align the function calling documentation with the version to be published on the Learn portal and remove any unused code.
microsoft · Nov 4, 2024 · e79ee3a · e79ee3a
1 parent 640fd91
commit e79ee3a
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 11 deletions.
diff --git a/dotnet/samples/Concepts/FunctionCalling/FunctionCalling.cs b/dotnet/samples/Concepts/FunctionCalling/FunctionCalling.cs
@@ -49,16 +49,16 @@ namespace FunctionCalling;
 ///    * The <see cref="FunctionChoiceBehaviorOptions.AllowParallelCalls"/> option instructs the AI model to call multiple functions in one request if the model supports parallel function calls.
 ///      By default, this option is set to null, meaning that the AI model default value will be used.
 ///
-///    The following table summarizes the effects of different combinations of these options:
+///    The following table summarizes the effects of various combinations of the AllowParallelCalls and AllowConcurrentInvocation options:
 ///
-///    | AllowParallelCalls  | AllowConcurrentInvocation | AI function call requests      | Concurrent Invocation |
-///    |---------------------|---------------------------|--------------------------------|-----------------------|
-///    | false               | false                     | one request per call           | false                 |
-///    | false               | true                      | one request per call           | false*                |
-///    | true                | false                     | one request per multiple calls | false                 |
-///    | true                | true                      | one request per multiple calls | true                  |
+///    | AllowParallelCalls  | AllowConcurrentInvocation | # of functions chosen per AI roundtrip  | Concurrent Invocation by SK |
+///    |---------------------|---------------------------|-----------------------------------------|-----------------------------|
+///    | false               | false                     | one                                     | false                       |
+///    | false               | true                      | one                                     | false*                      |
+///    | true                | false                     | multiple                                | false                       |
+///    | true                | true                      | multiple                                | true                        |
 ///
-///    `*` There's only one function to call
+///    `*` There's only one function to invoke.
 /// </summary>
 public class FunctionCalling(ITestOutputHelper output) : BaseTest(output)
 {

diff --git a/.../SemanticKernel.Abstractions/AI/FunctionChoiceBehaviors/RequiredFunctionChoiceBehavior.cs b/.../SemanticKernel.Abstractions/AI/FunctionChoiceBehaviors/RequiredFunctionChoiceBehavior.cs
@@ -81,12 +81,10 @@ public override FunctionChoiceBehaviorConfiguration GetConfiguration(FunctionCho
 
         var functions = base.GetFunctions(this.Functions, context.Kernel, this._autoInvoke);
 
-        IReadOnlyList<KernelFunction>? selectedFunctions = null;
-
         return new FunctionChoiceBehaviorConfiguration(this.Options ?? DefaultOptions)
         {
             Choice = FunctionChoice.Required,
-            Functions = selectedFunctions ?? functions,
+            Functions = functions,
             AutoInvoke = this._autoInvoke,
         };
     }