Skip to content

Conversation

Echo-Nie
Copy link
Contributor

@Echo-Nie Echo-Nie commented Apr 1, 2025

PR types

Others

PR changes

run.sh and README.md

Describe

run.sh中synthesize的vocoder是hifigan,synthesize_e2e的stage控制中文与英文的互相转换。在README.md中也进行相应补充。

#3997

Copy link

paddle-bot bot commented Apr 1, 2025

Thanks for your contribution!

@Echo-Nie
Copy link
Contributor Author

Echo-Nie commented Apr 1, 2025

@zxcd pls review

@luotao1 luotao1 added the HappyOpenSource 快乐开源活动issue与PR label Apr 3, 2025
@Echo-Nie Echo-Nie changed the title 【PaddleSpeech No.5】补全合成系列中的脚本中参数缺失 【PaddleSpeech No.5】 Apr 4, 2025
Copy link
Collaborator

@zxcd zxcd left a comment

Choose a reason for hiding this comment

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

check this

./run.sh --stage 3 --stop-stage 3 --gpus 0
```
`stage 3` of `run.sh` calls `local/synthesize_e2e.sh`.
`stage 3` of `run.sh` calls `local/synthesize_e2e.sh`, `stage 0` of it is **Speech cross language from en to zh** and `stage 1` of it is **Speech cross language from zh to en**.
Copy link
Collaborator

Choose a reason for hiding this comment

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

this part we have two stages in run.sh and synthesize_e2e.sh, the description need to be clearer

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

@Echo-Nie
Copy link
Contributor Author

@zxcd pls review

./run.sh --stage 3 --stop-stage 3 --gpus 0
```
`stage 3` of `run.sh` calls `local/synthesize_e2e.sh`.
`run.sh`'s `stage 3` invokes `synthesize_e2e.sh` and uses the `--stage` to select the specific task for end-to-end speech synthesis. By default, `synthesize_e2e.sh` executes `stage 0`, which performs cross-language speech synthesis from en to zh. To switch to cross-language speech synthesis from zh to en, you can use the `--stage 1` .
Copy link
Collaborator

Choose a reason for hiding this comment

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

default include both en-zh and zh-en?

Copy link
Collaborator

Choose a reason for hiding this comment

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

这段文字感觉可以描述的更清晰点,这里下面描述的参数都是synthesize_e2e.sh中包含的,这里是否可以不出现run.sh的使用介绍,直接介绍synthesize_e2e.sh呢?

类似这个文件的 Synthesizing 部分,直接介绍一下synthesize_e2e.sh 是否会更加清晰?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

Copy link
Contributor Author

Choose a reason for hiding this comment

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

`stage 3` of `run.sh` calls `local/synthesize_e2e.sh`.

我觉得第一句这个可以保留?说明 stage3 是调用 synthesize_e2e ,后面的一大串都是对 synthesize_e2e 的详细解释,会不会好一些

Copy link
Collaborator

Choose a reason for hiding this comment

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

可以的

@Echo-Nie Echo-Nie requested a review from zxcd April 15, 2025 12:31
Copy link
Collaborator

@zxcd zxcd left a comment

Choose a reason for hiding this comment

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

LGTM

@zxcd zxcd merged commit 2da5175 into PaddlePaddle:develop Apr 16, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants