dotnet / ai-samples

MIT License
399 stars 76 forks source link

semantic kernel execise - feedback #70

Open LittleLittleCloud opened 5 months ago

LittleLittleCloud commented 5 months ago

Sample 1

Spell error in initilization image

Sample 2

Sample 3

Sample 4

Sample 5

Sample 6

Sample 7

zewditu commented 5 months ago

image

Sample 4 does not have "using Microsoft.SemanticKernel.Connectors.OpenAI" these are using statements in sample 4

using Microsoft.Extensions.DependencyInjection;
using Microsoft.Extensions.Logging;
using Microsoft.SemanticKernel;
using Microsoft.SemanticKernel.ChatCompletion;
zewditu commented 5 months ago

For this I do not think you are not on latest image

zewditu commented 5 months ago

The LLM invokes the funciton call before I even start asking my age when I use gpt-3.5-turbo image

This is OpenAI model specific thing, if we add a persona like chatHistory.AddSystemMessage("Respond only to requested information. If the user says hi just respond greeting do not tell extra information"); It responds only greeting unless otherwise we ask age, but the main goal of this sample is to match Stephen's video not beyond that. Hence, I will update the prompt to the same prompt pattern of Stephen My name is Alice. How old am I? https://youtu.be/jrNfKeGSuCg?t=553

Naïve guess: the reason it responds age when name is provided because its calling behavior is set to AutoInvokeKernelFunctions.