Fastspeech2

[manmay-nakhashi]

added compute energy module

[erogol]

Thanks for the ✨PR✨. This is awesome.

We need some tests for EnergyDataset and the model here

Also wondering, have you had a chance to train the model for real?

[erogol]

Any plans on con'ting this PR?

[manmay-nakhashi]

@erogol should i add vits changes in same pr ?

[erogol]

@manmay-nakhashi better if you can send a separate PR for it.

[manmay-nakhashi]

@erogol i have resolved the conflict any thing else i need to do ?

[erogol]

@manmay-nakhashi thanks for updating the PR I'll try training the model and see how it works before going deeper in the code.

[erogol]

on to the comments above, we need testing under tests/tts_tests/ for the Fastspeech2 model. You can check the FastPitch model as an example.

[erogol]

There are still tests that do not pass :(

[erogol]

if you run make style you would fix the remaining style issue.

[manmay-nakhashi]

styles has been fixed , tests are failing now because of yourtts model in zoo

[erogol]

happens sometimes due to network incosistency. I restarted the failed jobs

[shivammehta25]

@erogol Did you try training this model or is something left to integrate this model?

[erogol]

I haven't had a chance to try the model. I don't know when I can do that but I'll do asap if someone else doesn't do it earlier.

[erogol]

I'll just merge for now and create follow up issues about the training.