Skip to content

Update blocks.py#33

Open
ksasso1028 wants to merge 2 commits intoHarmonai-org:mainfrom
ksasso1028:patch-1
Open

Update blocks.py#33
ksasso1028 wants to merge 2 commits intoHarmonai-org:mainfrom
ksasso1028:patch-1

Conversation

@ksasso1028
Copy link
Copy Markdown

@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
Copy Markdown
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
Copy Markdown
Author

ksasso1028 commented Apr 29, 2024

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

@opasquetdotfr
Copy link
Copy Markdown

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