Skip to content
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

[Hackathon 7th] tts3 with CSMSC #3921

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

Liyulingyue
Copy link
Contributor

PR types

Others

PR changes

Others

Describe

Enable pir infer

Copy link

paddle-bot bot commented Dec 1, 2024

Thanks for your contribution!

@mergify mergify bot added the T2S label Dec 1, 2024
@@ -126,6 +127,9 @@ def main():

paddle.set_device(args.device)

# set model_suffix
model_suffix = ".json" if os.path.exists(args.am + ".json") else ".pdmodel"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same comment with #3923

@Liyulingyue
Copy link
Contributor Author

Liyulingyue commented Dec 5, 2024

这边也按照 https://github.com/PaddlePaddle/PaddleSpeech/pull/3927/files 去改,动的太深了,建议不改 @zxcd

@zxcd
Copy link
Collaborator

zxcd commented Dec 6, 2024

refer #3927
fix interface def get_predictor in paddlespeech/t2s/exps/syn_utils.py

@Liyulingyue
Copy link
Contributor Author

Liyulingyue commented Dec 8, 2024

refer #3927 fix interface def get_predictor in paddlespeech/t2s/exps/syn_utils.py

#3927 paddlespeech/t2s/exps/speedyspeech/inference.py 是一个较为局限性的脚本,但是本PR涉及到的 paddlespeech/t2s/exps/syn_utils.py 是通用脚本,因此不便于参考 #3927 一样指定为固定的 str(Path(args.inference_dir)), "speedyspeech")。最好是加一个参数用于传输模型名称

@zxcd
Copy link
Collaborator

zxcd commented Dec 9, 2024

refer #3927 fix interface def get_predictor in paddlespeech/t2s/exps/syn_utils.py

#3927 paddlespeech/t2s/exps/speedyspeech/inference.py 是一个较为局限性的脚本,但是本PR涉及到的 paddlespeech/t2s/exps/syn_utils.py 是通用脚本,因此不便于参考 #3927 一样指定为固定的 str(Path(args.inference_dir)), "speedyspeech")。最好是加一个参数用于传输模型名称

是这样,所以你需要修改syn_utils.py get_predictor的接口,和对应的使用。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants