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.

Sign up for GitHub

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

BinConv2D for group convolution #6

Open

haithanhp opened this issue Feb 8, 2019 · 0 comments

haithanhp commented Feb 8, 2019

Dear @cooooorn ,

Thanks for your helpful implementation. I have 2 following concerns about class BinConv2d:

This line: self.weight = nn.Parameter(torch.IntTensor(out_channels, 1 + ( in_channels * self.kernel_size[0] * self.kernel_size[1] - 1) // 32)). Why do we divide 32 in testing process? I notice that the number of weights in testing is reduces by 32. Could you clarify that?
I want to use the group convolution. How can I modify for BinConv2d?
Thank you very much.

Thanks,
Hai

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment