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

Specifying the output size in transforms.GaussianBlur #8245

Open
talcs opened this issue Feb 1, 2024 · 4 comments
Open

Specifying the output size in transforms.GaussianBlur #8245

talcs opened this issue Feb 1, 2024 · 4 comments

Comments

@talcs
Copy link
Contributor

talcs commented Feb 1, 2024

📚 The doc issue

The transforms.GaussianBlur constructor takes kernel_size as a parameter. However, it isn't specified whether the convolution will use padding to keep the output image in the same dimension as the input. I think it should be mentioned. Maintaining the original input size is crucial in many cases, like object detection or segmentation, where the GT refers to certain pixels.

Suggest a potential alternative/fix

No response

@NicolasHug
Copy link
Member

Thanks for the report @talcs . GaussianBlur will indeed use padding:

output = torch_pad(output, padding, mode="reflect")

If you're interested, we'd be happy to review a PR from you to clarify that in the docstring

@Bhavay-2001
Copy link
Contributor

Hi @NicolasHug, I would be happy to work on this issue. Is it open for beginners?
Thanks

@NicolasHug
Copy link
Member

Thanks for your interest @Bhavay-2001 , but this has been taken care of in #8246 already.

@Bhavay-2001
Copy link
Contributor

Hi @NicolasHug, is there any other issue that I can contribute to?
Thanks

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

3 participants