autogen icon indicating copy to clipboard operation
autogen copied to clipboard

Removes Support For `TransformChatHistory` and `CompressibleAgent`

Open WaelKarkoub opened this issue 1 year ago • 2 comments

Why are these changes needed?

Doing some cleanup, removing TransformChatHistory and CompressibleAgent in favor of TransformMessages.

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.

WaelKarkoub avatar Aug 07 '24 01:08 WaelKarkoub

Codecov Report

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

Project coverage is 21.98%. Comparing base (6279247) to head (246015e). Report is 80 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #3313       +/-   ##
===========================================
- Coverage   32.90%   21.98%   -10.92%     
===========================================
  Files          94       95        +1     
  Lines       10235    10637      +402     
  Branches     2193     2442      +249     
===========================================
- Hits         3368     2339     -1029     
- Misses       6580     8128     +1548     
+ Partials      287      170      -117     
Flag Coverage Δ
unittests 21.94% <ø> (-10.97%) :arrow_down:

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 Aug 07 '24 01:08 codecov-commenter

Thanks @WaelKarkoub, should we remove this section from the FAQ?

Rest looks good to me :)

marklysze avatar Aug 08 '24 08:08 marklysze