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

doc: fix examples in README #10

Merged
merged 1 commit into from Sep 21, 2022
Merged

doc: fix examples in README #10

merged 1 commit into from Sep 21, 2022

Conversation

seriar
Copy link
Contributor

@seriar seriar commented Sep 21, 2022

Problem

The one-time snippet does not work as is and complains that the speaker is not defined

 > initialization of speaker-embedding layers.
 > Text: Перевірка мікрофона
 > Text splitted to sentences.
['Перевірка мікрофона']
Traceback (most recent call last):
  File "/home/serg/.local/bin/tts", line 8, in <module>
    sys.exit(main())
  File "/home/serg/.local/lib/python3.8/site-packages/TTS/bin/synthesize.py", line 350, in main
    wav = synthesizer.tts(
  File "/home/serg/.local/lib/python3.8/site-packages/TTS/utils/synthesizer.py", line 228, in tts
    raise ValueError(
ValueError:  [!] Look like you use a multi-speaker model. You need to define either a `speaker_name` or a `speaker_wav` to use a multi-speaker model.

Also it speakers.pth should be downloaded.

Fix

Just a few documentation changes:

  • make instructions on what to download from Releases more precise
  • add --speaker_id argument with one of the speakers

- make instructions on what to download from Releases more precise
- add `--speaker_id` argument with one of the speakers
@seriar
Copy link
Contributor Author

seriar commented Sep 21, 2022

@robinhad Hi, here is a suggestion on how to improve the documentation. The original snippet for a one-time run did not work for me and I had to do some changes to make it work. It could be helpful for others

@robinhad robinhad merged commit 215e4d0 into robinhad:main Sep 21, 2022
@robinhad
Copy link
Owner

Thank you @seriar! I also believe this whole section needs to be updated to highlight that you will use model without normalizer and autostress feature.

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

Successfully merging this pull request may close these issues.

None yet

2 participants