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.

Sign up for GitHub

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

Jump to bottom

A TTS recipe VITS on VCTK dataset #1380

Merged
merged 43 commits into from
Dec 6, 2023
Merged

A TTS recipe VITS on VCTK dataset #1380

merged 43 commits into from
Dec 6, 2023

Conversation

JinZr
Copy link
Collaborator

@JinZr JinZr commented Nov 10, 2023

akin to PR #1372

@yaozengwei
Copy link
Collaborator

If possible, could make files be soft links from the ljspeech recipe instead of copy, like model files. They are compatible for multi-speakers TTS.

@JinZr
Copy link
Collaborator Author

JinZr commented Nov 29, 2023

If possible, could make files be soft links from the ljspeech recipe instead of copy, like model files. They are compatible for multi-speakers TTS.

Noted, thanks!

I plan to replace duplicated files with softlinks after #1372 is merged to master branch.

yaozengwei
yaozengwei reviewed Dec 5, 2023
View reviewed changes
egs/vctk/TTS/prepare.sh Outdated Show resolved Hide resolved
requirements_tts.txt Outdated Show resolved Hide resolved
@JinZr JinZr mentioned this pull request Dec 6, 2023
Closed
@JinZr JinZr merged commit 735fb9a into k2-fsa:master Dec 6, 2023
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yaozengwei yaozengwei yaozengwei approved these changes

@csukuangfj csukuangfj Awaiting requested review from csukuangfj

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@JinZr @yaozengwei