dusty-nv / jetson-containers

Machine Learning Containers for NVIDIA Jetson and JetPack-L4T
MIT License
2.13k stars 440 forks source link

Add `whisperx` package #322

Closed tokk-nv closed 7 months ago

tokk-nv commented 10 months ago

Adding whisperx, faster-whisper and ctraslate2 packages.

johnnynunez commented 10 months ago

@dusty-nv I've check it and works well. Only one thing is that it has been carefully with ctranslate2 build python packages with the directory to build it. Or fixed with:

include_dirs = [pybind11.get_include(), '/usr/local/include/ctranslate2']
library_dirs = ['/usr/local/lib']

Or fixed with change: CMAKE_INSTALL_PREFIX from https://opennmt.net/CTranslate2/installation.html#install-from-sources

tokk-nv commented 10 months ago

Hi @johnnynunez Thank you for checking. Will you explain to me what you are proposing regarding the CTranslate2 build? I believe it found cmake fine, and it also built the Python wrapper.

johnnynunez commented 10 months ago

Hi @johnnynunez Thank you for checking. Will you explain to me what you are proposing regarding the CTranslate2 build? I believe it found cmake fine, and it also built the Python wrapper.

Yes because you use containers. If it's compile outside using for example miniconda, there is an error, from Ctranslate2 repository.

tokk-nv commented 9 months ago

ok, thank you.