Adding callable summary_method support and enhancements to initiate_chats #1628

qingyun-wu · 2024-02-11T18:39:30Z

Why are these changes needed?

Support callable summary_method to enable users to define their own methods for generating summaries.
Allow repetitive recipient agents in sequential chats in initiate_chats (closing [Issue]: Allow repetitive agents in multiple sequential chats #1595)
Enhance the default behavior of "summary_method". The default is "last_msg". When set to None, it now returns an empty string as a summary (this change was implemented in this PR). This adjustment accommodates scenarios where a summary is not needed.

Related issue number

Checks

I've included any doc changes needed for https://microsoft.github.io/autogen/. See https://microsoft.github.io/autogen/docs/Contribute#documentation to build and test documentation locally.
I've added tests (if relevant) corresponding to the changes introduced in this PR.
I've made sure all auto checks have passed.

…hats (microsoft#1628) * initiate_chats enhancements * callable summary_method * summary method * summary method default * docstr * add timeout to slient pip install test * consolidate_chat_info * a_initiate_chat * AssertionError test * update tests --------- Co-authored-by: Eric Zhu <ekzhu@users.noreply.github.com>

qingyun-wu added 4 commits February 11, 2024 13:09

initiate_chats enhancements

64acda0

callable summary_method

d87921a

summary method

f66df38

summary method default

69fa176

qingyun-wu had a problem deploying to openai1 February 11, 2024 18:39 — with GitHub Actions Failure

qingyun-wu marked this pull request as ready for review February 11, 2024 18:39

qingyun-wu had a problem deploying to openai1 February 11, 2024 18:39 — with GitHub Actions Failure

Merge branch 'main' into summary

cac26bf

qingyun-wu had a problem deploying to openai1 February 11, 2024 18:39 — with GitHub Actions Failure

qingyun-wu had a problem deploying to openai1 February 11, 2024 18:40 — with GitHub Actions Failure

qingyun-wu requested review from gagb and ekzhu February 11, 2024 18:40

qingyun-wu had a problem deploying to openai1 February 12, 2024 01:48 — with GitHub Actions Failure

sonichi approved these changes Feb 12, 2024

View reviewed changes

update tests

dfaa118

qingyun-wu had a problem deploying to openai1 February 12, 2024 01:51 — with GitHub Actions Failure

qingyun-wu temporarily deployed to openai1 February 12, 2024 01:51 — with GitHub Actions Inactive

qingyun-wu enabled auto-merge February 12, 2024 02:24

qingyun-wu added this pull request to the merge queue Feb 12, 2024

Merged via the queue into main with commit 899b250 Feb 12, 2024
59 of 60 checks passed

sonichi deleted the summary branch February 12, 2024 02:36

qingyun-wu had a problem deploying to openai1 February 12, 2024 02:40 — with GitHub Actions Failure

qingyun-wu temporarily deployed to openai1 February 12, 2024 02:40 — with GitHub Actions Inactive

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Adding callable summary_method support and enhancements to initiate_chats #1628

Adding callable summary_method support and enhancements to initiate_chats #1628

qingyun-wu commented Feb 11, 2024 •

edited by sonichi

Loading

Adding callable summary_method support and enhancements to initiate_chats #1628

Adding callable summary_method support and enhancements to initiate_chats #1628

Conversation

qingyun-wu commented Feb 11, 2024 • edited by sonichi Loading

Why are these changes needed?

Related issue number

Checks

qingyun-wu commented Feb 11, 2024 •

edited by sonichi

Loading