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

[Bug] " No module named 'TTS.tts.utils.text.symbols" #2574

Closed
beew opened this issue May 2, 2023 · 2 comments · Fixed by #3230
Closed

[Bug] " No module named 'TTS.tts.utils.text.symbols" #2574

beew opened this issue May 2, 2023 · 2 comments · Fixed by #3230
Labels
bug Something isn't working wontfix This will not be worked on but feel free to help.

Comments

@beew
Copy link

beew commented May 2, 2023

Describe the bug

I am trying to run this note book https://github.com/coqui-ai/TTS/blob/dev/notebooks/ExtractTTSpectrogram.ipynb

and it failed in the first cell with

" No module named 'TTS.tts.utils.text.symbols"

To Reproduce

run the note book

Expected behavior

It should run.

Logs

No response

Environment

TTs version 0.11.1

pytorch version 2.0.0+cpu

python 3.10.9

No cuda

OS Ubuntu 22.04

Additional context

No response
@beew beew added the bug Something isn't working label May 2, 2023
@erogol
Copy link
Member

erogol commented May 8, 2023

Notebooks are out of sync, unfortunately.

@stale
Copy link

stale bot commented Jun 9, 2023

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. You might also look our discussion channels.

@stale stale bot added the wontfix This will not be worked on but feel free to help. label Jun 9, 2023
@stale stale bot closed this as completed Jun 16, 2023
eginhard added a commit to idiap/coqui-ai-TTS that referenced this issue Nov 15, 2023
@eginhard
fix(ExtractTTSpectrogram.ipynb): adapt to current TTS code
ddbaecd 
Fixes coqui-ai#2447, coqui-ai#2574
@eginhard eginhard mentioned this issue Nov 15, 2023
Merged
erogol pushed a commit that referenced this issue Nov 16, 2023
@eginhard
Remove duplicate AudioProcessor code and fix ExtractTTSpectrogram.ipy…
3c2d5a9 
…nb (#3230)

* chore: remove unused argument

* refactor(audio.processor): remove duplicate stft+griffin_lim

* chore(audio.processor): remove unused compute_stft_paddings

Same function available in numpy_transforms

* refactor(audio.processor): remove duplicate db_to_amp

* refactor(audio.processor): remove duplicate amp_to_db

* refactor(audio.processor): remove duplicate linear_to_mel

* refactor(audio.processor): remove duplicate mel_to_linear

* refactor(audio.processor): remove duplicate build_mel_basis

* refactor(audio.processor): remove duplicate stft_parameters

* refactor(audio.processor): use pre-/deemphasis from numpy_transforms

* refactor(audio.processor): use rms_volume_norm from numpy_transforms

* chore(audio.processor): remove duplicate assert

Already checked in numpy_transforms.compute_f0

* refactor(audio.processor): use find_endpoint from numpy_transforms

* refactor(audio.processor): use trim_silence from numpy_transforms

* refactor(audio.processor): use volume_norm from numpy_transforms

* refactor(audio.processor): use load_wav from numpy_transforms

* fix(bin.extract_tts_spectrograms): set quantization bits

* fix(ExtractTTSpectrogram.ipynb): adapt to current TTS code

Fixes #2447, #2574

* refactor(audio.processor): remove duplicate quantization methods
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working wontfix This will not be worked on but feel free to help.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remove duplicate AudioProcessor code, fix ExtractTTSpectrogram.ipynb idiap/coqui-ai-TTS
2 participants
@erogol @beew