Reasoning Agent can execute code now. #545
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Collaborator
|
@BabyCNM thanks for adding execution support to the reasoning agent, have you tested this locally? |
Collaborator
Author
|
@Hk669 Yes, I have tested locally. However, I want to include tool_use too. Hold on a bit. |
|
|
alecsolder
approved these changes
Feb 25, 2025
Codecov ReportAll modified and coverable lines are covered by tests ✅
|
BabyCNM
added a commit
that referenced
this pull request
Feb 25, 2025
* Enable code execution in reasoning agent 1. enable code execution 2. remove the "verbose" parameter, and replace with AG2's default "silent" parameter. * init _user_proxy * notebook update * notebook update * add test case for running code in reasoning agent * use mock credential for reasoning test * reasoning prompt update * mock credentials for more tests * Update .secrets.baseline * clear notebook outputs * remove variable F
BabyCNM
added a commit
that referenced
this pull request
Feb 26, 2025
* Enable code execution in reasoning agent 1. enable code execution 2. remove the "verbose" parameter, and replace with AG2's default "silent" parameter. * init _user_proxy * notebook update * notebook update * add test case for running code in reasoning agent * use mock credential for reasoning test * reasoning prompt update * mock credentials for more tests * Update .secrets.baseline * clear notebook outputs * remove variable F
BabyCNM
added a commit
that referenced
this pull request
Feb 26, 2025
* Enable code execution in reasoning agent 1. enable code execution 2. remove the "verbose" parameter, and replace with AG2's default "silent" parameter. * init _user_proxy * notebook update * notebook update * add test case for running code in reasoning agent * use mock credential for reasoning test * reasoning prompt update * mock credentials for more tests * Update .secrets.baseline * clear notebook outputs * remove variable F
BabyCNM
added a commit
that referenced
this pull request
Feb 26, 2025
* Enable code execution in reasoning agent 1. enable code execution 2. remove the "verbose" parameter, and replace with AG2's default "silent" parameter. * init _user_proxy * notebook update * notebook update * add test case for running code in reasoning agent * use mock credential for reasoning test * reasoning prompt update * mock credentials for more tests * Update .secrets.baseline * clear notebook outputs * remove variable F
BabyCNM
added a commit
that referenced
this pull request
Feb 26, 2025
* Enable code execution in reasoning agent 1. enable code execution 2. remove the "verbose" parameter, and replace with AG2's default "silent" parameter. * init _user_proxy * notebook update * notebook update * add test case for running code in reasoning agent * use mock credential for reasoning test * reasoning prompt update * mock credentials for more tests * Update .secrets.baseline * clear notebook outputs * remove variable F
github-merge-queue bot
pushed a commit
that referenced
this pull request
Feb 28, 2025
…ningAgent (#1140) * Add prompt rewriting for multiple messages * Add tests for prompt handling * Reasoning Agent can execute code now. (#545) * Enable code execution in reasoning agent 1. enable code execution 2. remove the "verbose" parameter, and replace with AG2's default "silent" parameter. * init _user_proxy * notebook update * notebook update * add test case for running code in reasoning agent * use mock credential for reasoning test * reasoning prompt update * mock credentials for more tests * Update .secrets.baseline * clear notebook outputs * remove variable F * Add tests for prompt handling * Remove duplicate test functions from auto-merge. * Update rewrite prompt * Reasoning Agent can execute code now. (#545) * Enable code execution in reasoning agent 1. enable code execution 2. remove the "verbose" parameter, and replace with AG2's default "silent" parameter. * init _user_proxy * notebook update * notebook update * add test case for running code in reasoning agent * use mock credential for reasoning test * reasoning prompt update * mock credentials for more tests * Update .secrets.baseline * clear notebook outputs * remove variable F * Swarm and Google Gemini client test fixes (#1139) * Fix swarm tests by including transit messages * Fix for Gemini library for async use * pre-commit fix * Add tests for prompt handling * Reasoning Agent can execute code now. (#545) * Enable code execution in reasoning agent 1. enable code execution 2. remove the "verbose" parameter, and replace with AG2's default "silent" parameter. * init _user_proxy * notebook update * notebook update * add test case for running code in reasoning agent * use mock credential for reasoning test * reasoning prompt update * mock credentials for more tests * Update .secrets.baseline * clear notebook outputs * remove variable F * Reasoning Agent can execute code now. (#545) * Enable code execution in reasoning agent 1. enable code execution 2. remove the "verbose" parameter, and replace with AG2's default "silent" parameter. * init _user_proxy * notebook update * notebook update * add test case for running code in reasoning agent * use mock credential for reasoning test * reasoning prompt update * mock credentials for more tests * Update .secrets.baseline * clear notebook outputs * remove variable F * Add tests for prompt handling * Reasoning Agent can execute code now. (#545) * Enable code execution in reasoning agent 1. enable code execution 2. remove the "verbose" parameter, and replace with AG2's default "silent" parameter. * init _user_proxy * notebook update * notebook update * add test case for running code in reasoning agent * use mock credential for reasoning test * reasoning prompt update * mock credentials for more tests * Update .secrets.baseline * clear notebook outputs * remove variable F * merge * Resolve merge conflicts * Update rewrite prompt This rewriting: 1) generalizes 'STEPS ALREADY EXECUTED' to 'ACTIVITY LOG', and 2) generalizes 'current/last question asked by the user' to 'current/last question to be addressed' * Fix f-string bug * Remove out of prompt format line --------- Co-authored-by: giorgossideris <[email protected]> Co-authored-by: Davor Runje <[email protected]> Co-authored-by: Mark Sze <[email protected]> Co-authored-by: Davor Runje <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Why are these changes needed?
Related issue number
Checks