We use a common chat builder for all our applications that creates a FunctionInvokingChatClient. Many of these applications don't use tools but still get a FunctionInvokingChatClient.GetResponseAsync.
|
// A single request into this GetResponseAsync may result in multiple requests to the inner client. |
|
// Create an activity to group them together for better observability. |
|
using Activity? activity = _activitySource?.StartActivity($"{nameof(FunctionInvokingChatClient)}.{nameof(GetResponseAsync)}"); |
Would it make sense to suppress that span if options.Tools and AdditionalTools are empty?
We use a common chat builder for all our applications that creates a FunctionInvokingChatClient. Many of these applications don't use tools but still get a
FunctionInvokingChatClient.GetResponseAsync.extensions/src/Libraries/Microsoft.Extensions.AI/ChatCompletion/FunctionInvokingChatClient.cs
Lines 233 to 235 in da94b60
Would it make sense to suppress that span if
options.ToolsandAdditionalToolsare empty?