search

Ayanaminn / N46Whisper

Whisper based Japanese subtitle generator
MIT License
1.54k stars 127 forks source link

use fast-whisper with large-v3 #76

Closed ac1982 closed 8 months ago

ac1982 commented 8 months ago

added v3 support

Ayanaminn commented 8 months ago

我有点没明白为什么能通过修改版本号来适配，可否请您指教 另外 if model_size == 'large-v3'这个条件 是之前faster-whisper还暂时不支持v3的时候，暂时通过官方api来使用v3 现在 Add support for whisper-large-v3 #565这个PR生效后faster-whisper已经支持v3所以可以直接通过改模型名来选择模型了，所以也不需要这个条件了

ac1982 commented 8 months ago

是的，fast-whipser支持large-v3模型后，就不需要这个条件判断来做替代逻辑。但是我只修改版本号的原因，实际上起到了跳过的作用，但同时又保留了逻辑结构，这样做的好处是，未来如果引入whisper large-v4版本，我们只需调整版本号检查逻辑，而无需对其他代码进行修改。

Ayanaminn commented 8 months ago

原来如此，确实是种很巧妙的办法，学习了！