-
Notifications
You must be signed in to change notification settings - Fork 5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Allow SocietyOfMind agent to work with function calling and tools. #1569
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #1569 +/- ##
===========================================
+ Coverage 35.28% 65.30% +30.02%
===========================================
Files 44 44
Lines 5311 5324 +13
Branches 1231 1305 +74
===========================================
+ Hits 1874 3477 +1603
+ Misses 3283 1515 -1768
- Partials 154 332 +178
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
…icrosoft#1569) * Allow SocietyOfMind agent to work with function calling and tools. * Correctly skip OpenAI tests.
Why are these changes needed?
Resolves #1553, and adds tests to ensure it doesn't break again.
Checks