-
Notifications
You must be signed in to change notification settings - Fork 125
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
Number of flops #6
Comments
Because my implementation of group convolution is based on straightforward tensorflow graph operations. However, I think the authors used a fast cuDNN implementation based on caffe. |
Ok ! Thank you for your answer ! |
You're welcome. Also, note that the group convolution operator hasn't been implemented yet officially in Tensorflow. |
@Pelups I have an update to this. I found out that the authors have counted mult-add as 'one' unit not as 'two' units. So, the number I have achieved if roughly divided by two (assuming multiplications and additions have the same count), will give us their reported FLOPS. |
@MG2033 Would you share the "weights.pkl" |
Hi !
Does someone knows why the number of flops is about twice the number obtained by the authors ?
Thanks
The text was updated successfully, but these errors were encountered: