Skip to content

Update blocks.py #33

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Update blocks.py #33

wants to merge 2 commits into from

Conversation

ksasso1028
Copy link

@ksasso1028 ksasso1028 commented Apr 28, 2024

Allow Unet to process any arbitrary size (above kernel size). Just need to pad a few samples :)

Model should work with any sample length, as opposed to just with powers of 2 @zqevans

Allow Unet to process any arbitrary size (above kernel size)
@ksasso1028
Copy link
Author

ksasso1028 commented Apr 28, 2024

upon further inspection, @zqevans issue seems to be because you use a even kernel size for your fourier kernels in down/upscale, which means you arent padding enough using the typical padding formula for odd kernels ->(kernel - 1) // 2 * dil.

[1 / 8, 3 / 8, 3 / 8, 1 / 8],

@ksasso1028
Copy link
Author

ksasso1028 commented Apr 29, 2024

updated: removed the concat by accident on upload for git, have fixed

@opasquetdotfr
Copy link

Hello!

How and where would you decrease the "kernel size"?
Is there a way to train and inference using smaller arrays blocks than what is currently is?

Thanks!

O.

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

Successfully merging this pull request may close these issues.

2 participants