Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Implement alignment (from whisperx) #51

Merged
merged 20 commits into from
May 15, 2023
Merged

Implement alignment (from whisperx) #51

merged 20 commits into from
May 15, 2023

Conversation

chainyo
Copy link
Contributor

@chainyo chainyo commented May 15, 2023 •
edited
Loading

  • Implement alignment service (from whisperX repo)
  • Add an extra post-process to check all words have start/end values
  • Add alignment as a request parameter to choose to use it or not
  • Add experimental function for dealing with numbers to words for alignment (not used now). Need more work
  • Add some utils functions tests to up coverage score

@chainyo chainyo linked an issue May 15, 2023 that may be closed by this pull request
Add alignment service #38
Closed
@chainyo chainyo self-assigned this May 15, 2023
@chainyo chainyo added api Everything related to the API implementation transcription Everything related to the transcription part labels May 15, 2023
@chainyo chainyo merged commit 6e95d0f into main May 15, 2023
@chainyo chainyo deleted the 38-add-alignement-into-the-whisper-process branch May 15, 2023 11:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aleksandr-smechov aleksandr-smechov aleksandr-smechov approved these changes

Assignees

@chainyo chainyo

Labels
api Everything related to the API implementation transcription Everything related to the transcription part
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add alignment service
2 participants
@chainyo @aleksandr-smechov