Skip to content

WhisperX role: install whisperx inside virtualenv#1790

Open
philmcmahon wants to merge 4 commits intomainfrom
fix-whisperx-fetch-script
Open

WhisperX role: install whisperx inside virtualenv#1790
philmcmahon wants to merge 4 commits intomainfrom
fix-whisperx-fetch-script

Conversation

@philmcmahon
Copy link
Copy Markdown
Contributor

@philmcmahon philmcmahon commented Mar 2, 2026

What does this change?

The existing AMIgo role to install whisperx is broken for recent versions of the Deep Learning AMI and whisperx. This fixes those problems by installing whisperx in a virtual environment. As part of this work I had to update the script we use to fetch the models, and disable unattended-upgrades for all bakes in AMIgo

Depends on #1813 and guardian/transcription-service#273

How to test

I've tested this with an AMI baked here https://amigo.code.dev-gutools.co.uk/recipes/cuda-ami-test and then deployed to transcription CODE service to verify that whisperx is working properly

@philmcmahon philmcmahon changed the title Bump torch and torchaudio to support latest whisperx version WhisperX role: Bump torch and torchaudio to support latest whisperx version Mar 2, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 2, 2026

@philmcmahon philmcmahon self-assigned this Mar 2, 2026
@philmcmahon philmcmahon added the maintenance Departmental tracking: maintenance work, not a fix or a feature label Mar 2, 2026
@philmcmahon philmcmahon marked this pull request as ready for review March 2, 2026 15:10
@philmcmahon philmcmahon requested a review from a team as a code owner March 2, 2026 15:10
@philmcmahon philmcmahon force-pushed the fix-whisperx-fetch-script branch from b350d22 to 2a4090a Compare March 2, 2026 15:10
jorgeazevedo
jorgeazevedo previously approved these changes Mar 2, 2026
@philmcmahon
Copy link
Copy Markdown
Contributor Author

thanks for the review of this @jorgeazevedo , I'm still having some problems with it so will move back to draft for now

@philmcmahon philmcmahon marked this pull request as draft March 12, 2026 11:20
@philmcmahon philmcmahon force-pushed the fix-whisperx-fetch-script branch from 2a4090a to 7b57a60 Compare March 12, 2026 11:21
@philmcmahon philmcmahon force-pushed the fix-whisperx-fetch-script branch 4 times, most recently from c4cfdb9 to 1d94acb Compare April 1, 2026 15:40
@philmcmahon philmcmahon changed the base branch from main to remove-unattended-upgrades April 1, 2026 15:41
@philmcmahon philmcmahon changed the title WhisperX role: Bump torch and torchaudio to support latest whisperx version WhisperX role: install whisperx inside virtualenv Apr 1, 2026
@philmcmahon philmcmahon marked this pull request as ready for review April 2, 2026 08:15
twrichards
twrichards previously approved these changes Apr 2, 2026
Base automatically changed from remove-unattended-upgrades to main April 2, 2026 11:29
@philmcmahon philmcmahon dismissed twrichards’s stale review April 2, 2026 11:29

The base branch was changed.

@philmcmahon philmcmahon enabled auto-merge April 2, 2026 11:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintenance Departmental tracking: maintenance work, not a fix or a feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants