Skip to content

Conversation

rich04lin
Copy link
Contributor

@rich04lin rich04lin commented Mar 14, 2025

PR types
Others

PR changes
Docs

Describe
补全合成系列中的脚本中参数缺失 No.5
修改:
examples/aishell3_vctk/ernie_sat/run.sh
examples/aishell3_vctk/ernie_sat/README.md

Copy link

paddle-bot bot commented Mar 14, 2025

Thanks for your contribution!

@rich04lin
Copy link
Contributor Author

@zxcd

@luotao1 luotao1 changed the title 【Doc】补全合成系列中的脚本中参数缺失 No.5 【PaddleSpeech No.5】补全合成系列中的脚本中参数缺失 Mar 17, 2025
@luotao1 luotao1 added the HappyOpenSource 快乐开源活动issue与PR label Mar 17, 2025
@@ -27,10 +27,11 @@ if [ ${stage} -le 1 ] && [ ${stop_stage} -ge 1 ]; then
fi

if [ ${stage} -le 2 ] && [ ${stop_stage} -ge 2 ]; then
# synthesize, vocoder is pwgan
CUDA_VISIBLE_DEVICES=${gpus} ./local/synthesize.sh ${conf_path} ${train_output_path} ${ckpt_name} || exit -1
# synthesize, vocoder is pwgan by default stage 0, stage 1 will use hifigan as vocoder
Copy link
Collaborator

@zxcd zxcd Mar 18, 2025

Choose a reason for hiding this comment

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

synthesize.sh only has stage 0, no need to change this?

fi

if [ ${stage} -le 3 ] && [ ${stop_stage} -ge 3 ]; then
CUDA_VISIBLE_DEVICES=${gpus} ./local/synthesize_e2e.sh ${conf_path} ${train_output_path} ${ckpt_name} || exit -1
# synthesize_e2e, vocoder is pwgan by default stage 0, stage 1 will use hifigan as vocoder
Copy link
Collaborator

Choose a reason for hiding this comment

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

stage 0 and stage 1 all use hifigan. The stage distinction should not be based on the vocoder.

@luotao1
Copy link
Collaborator

luotao1 commented Mar 20, 2025

@rich04lin
Copy link
Contributor Author

CI done

@luotao1
Copy link
Collaborator

luotao1 commented Mar 24, 2025

image

评审人的每个意见都要回复,其他PR同

```
`--stage` controls the vocoder model during synthesis, which can be `0` or `1`, use `pwgan` or `hifigan` model as vocoder.
Copy link
Collaborator

Choose a reason for hiding this comment

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

take a look at ./local/synthesize.sh, there are four stages.

@luotao1
Copy link
Collaborator

luotao1 commented Apr 16, 2025

close due to the following PR is merged:

@luotao1 luotao1 closed this Apr 16, 2025
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.

3 participants