Chris
Chris
I am investigating approaches to this issue
We are currently working on a big investment on memory and storage: https://github.com/microsoft/semantic-memory Currently, the strategy is to incorporate SQL access as part of this. Active discussion are underway on...
@glennc - I've worked with some customers who are subclassing `ChatHistoryKernelAgent` and using composition (an internal `ChatCompletionAgent`) to address custom requirements around telemetry and instruction management. This is simliar to...
Yes this is all good. I was discussing the `LogMessages` approach with Dmytro. I think we nailed the other aspects outlined in this article. Was there something specific on your...
@joslat - Great points, indeed. I recently did some analysis along these vectors as part of https://github.com/microsoft/semantic-kernel/issues/6783. I see you've discovered this issue and commented, thank you. I hesitate to...
@joslat - (visibility) I have recently created an issue for summarizer: https://github.com/microsoft/semantic-kernel/issues/6828 This issue is based on the current design; although, it might be adapted given some of the broader...
Couldn't `AzureAISearchMemory` be updated to support hybrid search? The appsettings could be updated to control the mode (hybrid or simple vector): `AzureAISearchConfig`. Since kernel-memory owns the schema definition, the search...
@Matt-Scheetz - You can examine this project to see how to integrate tesseract into kernel-memory: https://github.com/microsoft/chat-copilot Otherwise, Azure Forms Recognizer is supported if you add the configuration: https://github.com/microsoft/kernel-memory/blob/main/service/Service/appsettings.json#L338
This is the pattern Mark proposed on code-review. Apparently the `ChatHistory.AddMessage()` method you identified is extraneous (Toub); although, I'm not 100% clear on what changes are planned.
The equivalent is the `Add` method on `List`: 