-
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
Re-commit of #2222: Expanded speaker name matching during speaker selection #2267
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2267 +/- ##
===========================================
+ Coverage 37.94% 57.02% +19.07%
===========================================
Files 77 77
Lines 7784 7784
Branches 1667 1805 +138
===========================================
+ Hits 2954 4439 +1485
+ Misses 4580 2958 -1622
- Partials 250 387 +137
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Thanks for the update. |
OpenAI tests passed except a known flaky test: https://github.com/microsoft/autogen/actions/runs/8544596821/job/23410987268?pr=2267#step:7:137 |
Changes committed as per #2222 to a branch within autogen repository.
Please see #2222 for the reason and code changes / revisions for "Expanded speaker name matching during speaker selection".
Related issue number
This PR replaces #2222.
Checks