semantic-kernel icon indicating copy to clipboard operation
semantic-kernel copied to clipboard
verified publisher icon microsoft

Fix planner RememberFunctions method

Open awharrison-28 opened this issue 2 years ago • 0 comments

Motivation and Context

This PR fixes the text that gets embedded for remembering planner functions and clarifies the parameters being passed to SaveInformationAsync()

Description

  • text to embed is the fully function string
  • the key is the function name

Contribution Checklist

  • [ ] The code builds clean without any errors or warnings
  • [ ] The PR follows SK Contribution Guidelines (https://github.com/microsoft/semantic-kernel/blob/main/CONTRIBUTING.md)
  • [ ] The code follows the .NET coding conventions (https://learn.microsoft.com/dotnet/csharp/fundamentals/coding-style/coding-conventions) verified with dotnet format
  • [ ] All unit tests pass, and I have added new tests where possible
  • [ ] I didn't break anyone :smile:

awharrison-28 avatar Apr 17 '23 23:04 awharrison-28