autogen icon indicating copy to clipboard operation
autogen copied to clipboard

context to kwargs

Open qingyun-wu opened this issue 1 year ago • 1 comments

Why are these changes needed?

  • use kwargs instead of context
  • printing

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.
  • [x] I've added tests (if relevant) corresponding to the changes introduced in this PR.
  • [x] I've made sure all auto checks have passed.

qingyun-wu avatar Mar 18 '24 21:03 qingyun-wu

Codecov Report

Attention: Patch coverage is 86.36364% with 3 lines in your changes are missing coverage. Please review.

Project coverage is 56.44%. Comparing base (af9b300) to head (1ea407d).

Files Patch % Lines
autogen/agentchat/conversable_agent.py 83.33% 3 Missing :warning:
Additional details and impacted files
@@             Coverage Diff             @@
##             main    #2064       +/-   ##
===========================================
+ Coverage   36.77%   56.44%   +19.67%     
===========================================
  Files          69       69               
  Lines        7133     7134        +1     
  Branches     1556     1685      +129     
===========================================
+ Hits         2623     4027     +1404     
+ Misses       4277     2765     -1512     
- Partials      233      342      +109     
Flag Coverage Δ
unittests 56.34% <86.36%> (+19.59%) :arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov-commenter avatar Mar 18 '24 21:03 codecov-commenter