Move executor diag print to conversable_agent #1867

jackgerrits · 2024-03-05T15:23:00Z

Why are these changes needed?

Executors should not be printing to terminal. This moves handling of this to the conversable agent instead.

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.

* print in conversable agent, not in executor * fix fstring

jackgerrits added 2 commits March 5, 2024 10:21

print in conversable agent, not in executor

fc48804

fix fstring

af15c93

jackgerrits requested a review from ekzhu March 5, 2024 15:23

ekzhu approved these changes Mar 5, 2024

View reviewed changes

ekzhu added this pull request to the merge queue Mar 5, 2024

Merged via the queue into microsoft:main with commit 09a4918 Mar 5, 2024
46 of 57 checks passed

jackgerrits had a problem deploying to openai1 April 4, 2024 15:23 — with GitHub Actions Failure

whiskyboy pushed a commit to whiskyboy/autogen that referenced this pull request Apr 17, 2024

Move executor diag print to conversable_agent (microsoft#1867)

9123070

* print in conversable agent, not in executor * fix fstring

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Move executor diag print to conversable_agent #1867

Move executor diag print to conversable_agent #1867

jackgerrits commented Mar 5, 2024

Move executor diag print to conversable_agent #1867

Move executor diag print to conversable_agent #1867

Conversation

jackgerrits commented Mar 5, 2024

Why are these changes needed?

Related issue number

Checks