riverphoenix / tacotron2

Tacotron 2 implementation
87 stars 31 forks source link

get_spectrograms issue #9

Open LiNaihan opened 6 years ago

LiNaihan commented 6 years ago

Hi riverphoenix, this code is fantastic, but I think there may be some little error when generating mel spec using prepro.get_spectrograms. Since you get mag at line 31 with the param "power=1"(default), the mag should be the amplitude instead of power; so when you convert mag to decibel at line 41, you should not use "power_to_db", though the result differs just a factor of 2, I just want to discuss this issue with you and see if I'm wrong.

thx!