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.

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 WhisperX Role #1606

Merged
merged 3 commits into from
Jan 22, 2025
Merged

Add WhisperX Role #1606

merged 3 commits into from
Jan 22, 2025

Conversation

philmcmahon
Copy link
Contributor

What does this change?

Depends on #1604

WhisperX is a transcription tool - see github repo here https://github.com/m-bain/whisperX. It is potentially a lot faster than the tool we currently use for transcription, and supports speaker identification which is a feature often requested by our users.

This PR adds a role to AMIgo to install whisperx and associated dependencies.

How to test

Tested on CODE - I baked an AMI and then booted an instance with it and verified that whisperx worked

@philmcmahon philmcmahon requested a review from a team as a code owner January 17, 2025 14:53
jorgeazevedo
jorgeazevedo previously approved these changes Jan 17, 2025
@philmcmahon philmcmahon merged commit 65deded into main Jan 22, 2025
4 checks passed
@philmcmahon philmcmahon deleted the pm-whisperx-role branch January 22, 2025 16:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants