kernel-memory icon indicating copy to clipboard operation
kernel-memory copied to clipboard

[Question] does content.url in filename for websites make sense? (I want attribution per paragraph via separate prompt)

Open chaelli opened this issue 1 month ago • 2 comments

Context / Scenario

I changed the prompt to make sure the llm includes the source per paragraph of the answer. So I can more closly align the response with the facts for my users. When I do that, I can only tell it to reference the filename (as this is what the llm gets in the facts part of the prompt). For websites this is always "content.url" - because this is set so in https://github.com/microsoft/kernel-memory/blob/a1f280c42c4df9a60d1d5cecf0633d07ff927b1b/service/Core/MemoryService.cs#L120

Question

I wonder if it would not make more sense to put the url there instead of a static string. Or at least include the url in the facts where it exists.

chaelli avatar May 15 '24 06:05 chaelli