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

Hifigan Vocoder #16

Closed
wants to merge 19 commits into from
Closed

Hifigan Vocoder #16

wants to merge 19 commits into from

Conversation

rishikksh20
Copy link
Contributor

No description provided.

@erogol
Copy link
Member

erogol commented Mar 24, 2021

Any updates @rishikksh20 ?

@thorstenMueller
Copy link
Contributor

I'm interested in HifiGAN too. Excited to do a training on my "Thorsten" dataset.

@rishikksh20
Copy link
Contributor Author

@erogol Yes I have trained model on different configuration but not able to train successfully, I even copy paste original hifi-gan with exact settings and only use TTS data loader with pre-processing and yet result is same the audio is not understandable at all I think I need to go through TTS pre-processing thoroughly because I think I miss something there, also I have implemented l1 loss mentioned in hifigan paper and training on original hifigan architecture lets see how it's turnout. Also original hifigan takes a week to get trained so it takes time.

@erogol
Copy link
Member

erogol commented Mar 27, 2021

@rishikksh20 do you have a certain timeline on this? I try to plan the next release and see if you are up for it.

Feel free to tell if time is scarse, just asking for planing.

@rishikksh20
Copy link
Contributor Author

@erogol it will finished within end of first week of April.

@rishikksh20
Copy link
Contributor Author

@erogol One more think when I replaced TTS melspectrogram pre-processing with mine melspectrogram calculation it's working fine I think the problem is with pre-processing only.

@erogol
Copy link
Member

erogol commented Apr 13, 2021

I guess we can retire this as we merged HiFiGAN. Thanks, @rishikksh20 for initiating the work.

@erogol erogol closed this Apr 13, 2021
eginhard referenced this pull request in idiap/coqui-ai-TTS Apr 2, 2024
ci: combine coverage reports and merge test workflows
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants