-
Notifications
You must be signed in to change notification settings - Fork 372
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
Had various issues recently, likely related to packages #275
Comments
Hello, I added the first line to the notebook as it was needed for cuda compatibility, I don't think the others are needed anywhere |
I agree it's only
torch 2.4 and up installs I can open a PR with the following additions to
|
Cuda versions compatibility is not in the scope of requirements file, this is up for the user to decide, this project works with other torch versions than 2.3 so it should not be limited to 2.3 because of cuda constraints that are the user responsibility |
The script works fine for me with |
I had to install additional requirements that were not in any of the requirement files (except for the first one that is mentioned in this and some other issues): pip install "torch<2.4" "torchaudio<2.4" "ctranslate2<4.5"
pip search tensorrt_libs
pip install tensorrt
pip install nvidia-cuda-nvcc-cu12 |
I really don't understand how these packages are needed, they are not included because they are not used or needed, there is a CI action that runs periodically to test the needed requirements and it works out of the box with the current installation instructions |
Well, I guess I was rushing things and didn't pay much attention to what libraries you included in setting up LD_LIBRARY_PATH in another issue ticket:
|
Prepended the following to the install block in colab:
Most of these were googled, but @Leandrocnf helped out on this thread: m-bain/whisperX#902 (comment)
I don't know if this is worthy of a PR, but I wanted to share.
Thank you Mahmoud!
The text was updated successfully, but these errors were encountered: