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

180 degree rotated Correlation with Padding:full. #3

Open
abishekatp opened this issue Mar 17, 2023 · 2 comments
Open

180 degree rotated Correlation with Padding:full. #3

abishekatp opened this issue Mar 17, 2023 · 2 comments

Comments

@abishekatp
Copy link

Hi guys, Great work on this project. so we use ConvolutionLayer for calculating correlation for multiple input channel with multiple filter/kernels with corresponding channel.
How can I do the 180 degree rotated correlation(This is actually called convolution) with Padding:full option. As of now we have Same and Valid paddings.

@Conzel
Copy link
Owner

Conzel commented Mar 17, 2023

Hey! Thanks for the kind words :)

As far as I am aware, Full padding is not used in pytorch (whose API we want to be compatible with) as of now, so we wouldn’t implement it.

However, full padding should be realisable (in stride 1 at least) by setting the padding to the kernel size - 1.

Being able to provide integers for the padding is something I wanted to do for some time anyways, if you have a few days, I can try to implement it :)

@abishekatp
Copy link
Author

Thank you for answering. I was trying to use function like convolve2d in sci-py.
As you have said I think customizable padding will be helpful.

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