-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Fix a bug of streaming_tts_server #3865
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
Conversation
Fix "Failed to get model related files" when using fastspeech2_csmsc_onnx model in streaming_tts_server.
Which environment are you running in? This issue should not occur under the recommended configuration. |
环境:最新代码自己构建的镜像,以及 配置:将默认配置里的
改成另一个可用模型 启动 究其原因,选择 PaddleSpeech/paddlespeech/resource/pretrained_models.py Lines 1735 to 1744 in 6316514
这里的 PaddleSpeech/paddlespeech/server/engine/tts/online/onnx/tts_engine.py Lines 70 to 81 in 6316514
相对地,默认配置选择的是 PaddleSpeech/paddlespeech/resource/pretrained_models.py Lines 1795 to 1805 in 6316514
这里的 PaddleSpeech/paddlespeech/server/engine/tts/online/onnx/tts_engine.py Lines 94 to 108 in 6316514
所以有了这个 PR😊 |
好的,也麻烦把CI过一下,方便合入代码:D |
想问问上图这个CodeStyle,它建议是改成这样: 但是我仅仅把 PaddleSpeech/paddlespeech/server/engine/tts/online/onnx/tts_engine.py Lines 78 to 84 in 6316514
PaddleSpeech/paddlespeech/server/engine/tts/online/onnx/tts_engine.py Lines 101 to 114 in 6316514
感觉保持原样比较统一,还是说一定要过了才能合并呢😂😂😂 |
关于代码风格的问题,你可以使用pre-commit这个工具,使用参考:
|
LGTM. Thank you for your contribution! |
hi, @SuiYunsy
|
PR types
Bug fixes
PR changes
Others
Describe
Fix "Failed to get model related files" when using fastspeech2_csmsc_onnx model in streaming_tts_server.