autogen icon indicating copy to clipboard operation
autogen copied to clipboard

remove name from tool response

Open yenif opened this issue 1 year ago • 3 comments

Why are these changes needed?

#1255

Related issue number

Closes #1255

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.

yenif avatar Jan 15 '24 17:01 yenif

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (bde2fc9) 32.10% compared to head (d6f2e2a) 51.20%.

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1263       +/-   ##
===========================================
+ Coverage   32.10%   51.20%   +19.10%     
===========================================
  Files          32       32               
  Lines        4392     4394        +2     
  Branches     1024     1080       +56     
===========================================
+ Hits         1410     2250      +840     
+ Misses       2866     1941      -925     
- Partials      116      203       +87     
Flag Coverage Δ
unittests 51.13% <100.00%> (+19.07%) :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 Jan 15 '24 17:01 codecov-commenter

Thanks. Could you modify the test to address the test failures?

sonichi avatar Jan 15 '24 17:01 sonichi

@yenif the openai test is failing: https://github.com/microsoft/autogen/actions/runs/7532202265/job/20511225188?pr=1263#step:7:1189 Could you take a look this? Thank you!

qingyun-wu avatar Jan 16 '24 00:01 qingyun-wu