Skip to content

fix(faster-whisper): manually install ctranslate2 with rocm support before installing other dependencies - #9839

Open
eglia wants to merge 4 commits into
mudler:masterfrom
eglia:fix-rocm
Open

fix(faster-whisper): manually install ctranslate2 with rocm support before installing other dependencies#9839
eglia wants to merge 4 commits into
mudler:masterfrom
eglia:fix-rocm

Conversation

@eglia

@eglia eglia commented May 15, 2026

Copy link
Copy Markdown
Contributor

Description
faster-whisper uses ctranslate2, which by default does not have rocm support enabled, leading to the following error:

CUDA failed with error CUDA driver version is insufficient for CUDA runtime version

Unfortunately, ctranslate2 with enabled rocm support is only available from github releases. Therefore, this PR manually installs this version, before calling the regular installRequirements procedure.

Notes for Reviewers

Signed commits

  • Yes, I signed my commits.

…efore installing other dependencies

Signed-off-by: Andreas Egli <github@kharan.ch>
if [ "x${BUILD_PROFILE}" == "xhipblas" ]; then
ensureVenv
mkdir /tmp/ctranslate2-rocm
wget -O /tmp/ctranslate2-rocm/rocm-python-wheels-Linux.zip https://github.com/OpenNMT/CTranslate2/releases/download/v4.7.1/rocm-python-wheels-Linux.zip

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can we move the version/OS used as variables? so we can wire it up with the automatic bump workflow that we already have here: https://github.com/mudler/LocalAI/blob/master/.github/workflows/bump_deps.yaml

Assisted-by: opencode:gpt-5.5
Signed-off-by: Andreas Egli <github@kharan.ch>
@eglia

eglia commented May 19, 2026

Copy link
Copy Markdown
Contributor Author

I tried to integrate it into the bump_deps workflow similar to bump-vllm-wheel. Let me know if this is what you expected.

@eglia
eglia requested a review from mudler May 26, 2026 08:05

@localai-org-maint-bot localai-org-maint-bot left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mudler This looks good to merge. The ROCm wheel is limited to the hipblas profile, the pinned v4.7.1 release contains the expected Linux asset, and the bump workflow validates that asset before updating the pin. DCO is green.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants