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

[tts] update finetune yaml and readme #2418

Merged
merged 7 commits into from
Sep 20, 2022

Conversation

lym0302
Copy link
Contributor

@lym0302 lym0302 commented Sep 20, 2022

PR types
update

PR changes
update finetune yaml and readme

Describe
update finetune yaml and readme

@@ -221,3 +221,5 @@ optional arguments:

### Tips
If you want to get better audio quality, you can use more audios to finetune.
More finetune results can be viewed [finetune-fastspeech2-for-csmsc](https://paddlespeech.readthedocs.io/en/latest/tts/demo.html#finetune-fastspeech2-for-csmsc).
The results show the effect on csmsc_mini: frozen encoder > no frozen > frozen encoder and duration predictor.
Copy link
Collaborator

Choose a reason for hiding this comment

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

这块咱俩的写法要不要保持一致,我是这么写的

Freeze encoder > Non Frozen > Freeze encoder && duration_predictor

@@ -221,3 +221,5 @@ optional arguments:

### Tips
If you want to get better audio quality, you can use more audios to finetune.
More finetune results can be viewed [finetune-fastspeech2-for-csmsc](https://paddlespeech.readthedocs.io/en/latest/tts/demo.html#finetune-fastspeech2-for-csmsc).
Copy link
Collaborator

@yt605155624 yt605155624 Sep 20, 2022

Choose a reason for hiding this comment

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

can be viewed in

Copy link
Collaborator

@yt605155624 yt605155624 left a comment

Choose a reason for hiding this comment

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

LGTM

@yt605155624 yt605155624 merged commit 99551e5 into PaddlePaddle:develop Sep 20, 2022
@yt605155624 yt605155624 added this to the r1.2.0 milestone Sep 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

2 participants