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

ADR - Agent Framework

Open crickman opened this issue 1 year ago • 4 comments

Motivation and Context

This ADR captures agent concepts as the project solidifies. Will be updated as needed to remain in sync.

While the code exists as ground-truth, capturing the following additional vectors can aid with translation to Python and Java:

  • Conceptual overview
  • Objectives / requirements
  • Analysis
  • Intent

Description

This merges the initial Agent ADR with the current direction with the goal of capturing project essence for translation into Python and Java

Contribution Checklist

crickman avatar Apr 18 '24 17:04 crickman

Love seeing this :) That the ADR is again at full - thanks @crickman and team 💯

Questions: I am somehow missing the AutoGen Summarizer which I believe clarifies it very much in a "AutoGen Chat" is this obtained by ResultParser instead? It would be great to have a mapping of concepts and technologies like a chart...

Also mentioned on other PR but we need more in Agents for them to be usable business wise:

  • Two Agent Chat (or a simple variation of the groupChat)
  • Sequential Chat
  • Nested Chat (which can be recursive)
  • A layer of reliability supporting this - similar to Polly... but I'd like this to be in kind of a business workflow, so maybe a managed worklfow system..

Otherwise, a way to extend and support this would be great... as far as I understand the strategy is to support the basic constructs but enable the others to be extended on top so maybe we see this features in Autogen for .NET...

Thoughts?

Again, thanks @crickman & team!

joslat avatar Jun 26 '24 16:06 joslat

@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 get into the immediate discussion / decision-making around the concepts you've addressed...other than to say, I'm hopeful we'll have consensus at some point to address these.

crickman avatar Jun 26 '24 20:06 crickman

@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 issues being considered (https://github.com/microsoft/semantic-kernel/issues/6783).

crickman avatar Jun 27 '24 16:06 crickman

🍌

msbatin1 avatar Jun 27 '24 23:06 msbatin1

nice, only thing i am missing is the summarizer (maybe it's me, pretty late here...) but now looks pretty much complete and much more extensible 💯

joslat avatar Jul 22 '24 23:07 joslat