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

ExtractTTSpectrogram.ipynb not working #2447

Closed
t-dan opened this issue Mar 22, 2023 · 2 comments · Fixed by #3230
Closed

ExtractTTSpectrogram.ipynb not working #2447

t-dan opened this issue Mar 22, 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

@t-dan
Copy link

t-dan commented Mar 22, 2023

Describe the bug

Would it be possible to update ExtractTTSpectrogram.ipynb notebook to work again? It uses some non-existing modules and parameters, for example.

To Reproduce

Just run the notebook.

Expected behavior

No response

Logs

No response

Environment

{
    "CUDA": {
        "GPU": [],
        "available": false,
        "version": "11.3"
    },
    "Packages": {
        "PyTorch_debug": false,
        "PyTorch_version": "1.10.2+cu113",
        "TTS": "0.11.1",
        "numpy": "1.19.4"
    },
    "System": {
        "OS": "Linux",
        "architecture": [
            "64bit",
            "ELF"
        ],
        "processor": "x86_64",
        "python": "3.8.10",
        "version": "#1 SMP Debian 5.10.106-1+zs1 (2022-03-28)"
    }
}

Additional context

No response

@t-dan t-dan added the bug Something isn't working label Mar 22, 2023
@connershoop
Copy link

Please see this discussion, it may help: #2307 (reply in thread)

@stale
Copy link

stale bot commented Apr 21, 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 Apr 21, 2023
@stale stale bot closed this as completed Apr 28, 2023
eginhard added a commit to idiap/coqui-ai-TTS that referenced this issue Nov 15, 2023
erogol pushed a commit that referenced this issue Nov 16, 2023
…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
Labels
bug Something isn't working wontfix This will not be worked on but feel free to help.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants