-
Notifications
You must be signed in to change notification settings - Fork 5.3k
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
Add async test #520
Add async test #520
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #520 +/- ##
=======================================
Coverage 26.63% 26.63%
=======================================
Files 28 28
Lines 3777 3777
Branches 858 858
=======================================
Hits 1006 1006
Misses 2700 2700
Partials 71 71
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Fixed. And thanks Chi, will keep in mind for future PRs. |
* Add example nb for async funcs * Add a notebook based test for async function calls * Update nb * Update nb * Remove duplicate code * Rename func for consistency * Fix bug * Add intro text for cmd cell 4 * Add a short comment on await * Update agentchat_function_call_async.ipynb Minor typo * Add link to nb --------- Co-authored-by: Joshua Kim <[email protected]> Co-authored-by: Qingyun Wu <[email protected]>
* Add example nb for async funcs * Add a notebook based test for async function calls * Update nb * Update nb * Remove duplicate code * Rename func for consistency * Fix bug * Add intro text for cmd cell 4 * Add a short comment on await * Update agentchat_function_call_async.ipynb Minor typo * Add link to nb --------- Co-authored-by: Joshua Kim <[email protected]> Co-authored-by: Qingyun Wu <[email protected]>
Why are these changes needed?
Adds a test notebook for async function calls (including group chat)
Related issue number
#517
Checks