beeldengeluid / whisper-asr-worker

MIT License
2 stars 0 forks source link

Rewrite Dockerfile #67

Open gb-beng opened 2 months ago

gb-beng commented 2 months ago
jblom commented 2 months ago

@gb-beng we'll definitely wait with this one, since @greenw0lf mentioned having trouble with other setups. When we've tackled the deployment in ocp and using Airflow we can have a go with this one

gb-beng commented 2 months ago

Ok, I did test it locally and everything worked smoothly; as for visxp worker.

jblom commented 2 months ago

@gb-beng ah good news. Now it's a matter of seeing how it runs in ocp (I assume you did not run it there)

gb-beng commented 1 month ago

Some learnings:

You can manage CUDA libraries via pip, which I implemented in https://github.com/beeldengeluid/whisper-asr-worker/pull/67/commits/9630d88521e9135fb10d25fdbef29c0ad60cf103. I can run this version on the GPU but you'll need to manually set LD_LIBRARY_PATH, per https://github.com/SYSTRAN/faster-whisper/blob/d57c5b40b06e59ec44240d93485a95799548af50/README.md?plain=1#L101 to get it to work.

Because of this I think the nvidia cuda base images are a more solid approach, but I still would like to take out poetry to align with our other containers.


Notes: