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

Convolution operation can be replaced by MaxPool #16

Closed
miguelvr opened this issue Nov 29, 2018 · 3 comments
Closed

Convolution operation can be replaced by MaxPool #16

miguelvr opened this issue Nov 29, 2018 · 3 comments

Comments

@miguelvr
Copy link
Owner

Description:

A maxpool operation can be used for the block mask calculation and might be more efficient to compute than a convolution.

@linzehui
Copy link

have you figured out how to replace convolution with maxpooling operation? I found convolution operation very slow.

@linzehui
Copy link

I tested two operation, MaxPooling can be much faster than Convolution.
An example may be like that:
mask = F.max_pool2d(input=mask[:, None, :, :],
kernel_size= (block_size, block_size),
stride=(1,1),
padding=(block_size ,block_size))

@miguelvr
Copy link
Owner Author

I know how to do it, just don't have much time to do it. Can you do a PR?

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