-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
请求审核PR #3111
Comments
感谢贡献代码,在 review中了已经 |
overlap_ratio:表示重叠率(overlap ratio)或者叫做帧移率(hop size ratio),等于n_fft/hop_size,默认为4(相比于设置为1和2,在设置为4时,实验的合成效果更好),参考paddle istft hop_length |
pr 上提了两个意见,有空可以看下 |
好的,感谢 |
根据pr意见,均已经修改,目前CI的CodeStyle没有通过,但我force push 之前CI通过的代码,也没有通过CodeStyle,不知道是什么原因 |
我们复现的模型为C8C8I; 根据这三个参数应该在代码里面被算出来,我们将hop_size和n_fft设置为在代码中计算,overlap_ratio取4,来和论文中设置一致; 另外,我们也尝试了让overlap_ratio= 1, 2,合成的音质会有损失,我们认为这是因为nfft的减小带来了频率分辨率的降低影响了合成的音质。 训练模型是ok的,我们在对应的iSTFTNet.md中给出了50000次的hifigan和istftNet的预训练模型百度网盘链接和实验结果对比 |
能否请您审核我的 PR 并提供反馈?我将不胜感激。
#3006
The text was updated successfully, but these errors were encountered: