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

SpectrogramUpsampler only can be used when hop_samples=256 how to make it to support more hop_samples #44

Open
2651084156 opened this issue Jan 23, 2023 · 2 comments

Comments

@2651084156
Copy link

thecode in this repository only support hop_samples=256
图片

if i try change it i will get
图片
i try it to fix it
图片
but i think it not a good way to fix it
could i have other way to fix it and make it support more hop_samples

@sharvil
Copy link
Contributor

sharvil commented Jan 23, 2023

The hop size is tightly coupled with the upsampler module. You'll have to make changes to the transposed convolutions whenever you change the hop size. Here's an example of how you could change the hop size to 300: #6. In general, though, you'll have to compute the right arguments for the transposed convolutions.

@2651084156
Copy link
Author

The hop size is tightly coupled with the upsampler module. You'll have to make changes to the transposed convolutions whenever you change the hop size. Here's an example of how you could change the hop size to 300: #6. In general, though, you'll have to compute the right arguments for the transposed convolutions.

thank i fix it

and

if i want use sample_rate=44100
which Model params
i should make it bigger than Default params?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants