-
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
Update whisper notebook to use new function registration syntax #1918
Conversation
@qingyun-wu could you request review from the original contributor? |
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.
I ran the notebook, it works. One thing: set code_execution_config={}
for the user proxy agent to avoid warning.
Updated |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1918 +/- ##
==========================================
+ Coverage 37.13% 37.33% +0.19%
==========================================
Files 64 64
Lines 6794 6913 +119
Branches 1493 1519 +26
==========================================
+ Hits 2523 2581 +58
- Misses 4062 4109 +47
- Partials 209 223 +14
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
…osoft#1918) * Update notebook to use new function registration syntax * Update agentchat_video_transcript_translate_with_whisper.ipynb * formatting
Why are these changes needed?
Closes #1916
Related issue number
Checks