-
Notifications
You must be signed in to change notification settings - Fork 171
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
(feat) configurable timestamp options for audio-to-text #3207
(feat) configurable timestamp options for audio-to-text #3207
Conversation
02c608b
to
e7e633d
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## ai-video #3207 +/- ##
===================================================
- Coverage 36.07820% 35.92244% -0.15576%
===================================================
Files 124 124
Lines 34525 34658 +133
===================================================
- Hits 12456 12450 -6
- Misses 21381 21520 +139
Partials 688 688 see 1 file with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
da087f9
to
3807cdc
Compare
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.
Wait I think I broke something.
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 think I broke something.
This commit updates the ai-worker to the latest version so that users can start using the new `audio-to-text` `return_timestamps` field.
3807cdc
to
9d5130c
Compare
What does this pull request do? Explain your changes. (required)
This change adds the
return_timestamps
parameter to theaudio-to-text
pipeline, allowing end-users to configure the inference job to return timestamps at word-level, sentence-level or no timestamps at all.Supported values for
return_timestamps
arefalse
andword
. The pipeline defaults to existing behavior of sentence-level timestamp transcription to avoid breaking changes with existing applications.Specific updates (required)
How did you test each of these updates (required)
sentence-level timestamps
return_timestamps
parameter to verify inference job still defaults to sentence-level timestampssentence-timestamps.json
word-level timestamps
return_timestamps=word
to validate timestamps are returned at word-levelword-timestamps.json
no timestamps
return_timestamps=false
to validate timestamps are excludedno-timestamps.json
Does this pull request close any open issues?
AI-630
Checklist:
make
runs successfully./test.sh
pass