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

Number of flops #6

Open
Pelups opened this issue Nov 23, 2017 · 5 comments
Open

Number of flops #6

Pelups opened this issue Nov 23, 2017 · 5 comments

Comments

@Pelups
Copy link

Pelups commented Nov 23, 2017

Hi !
Does someone knows why the number of flops is about twice the number obtained by the authors ?

Thanks

@MG2033
Copy link
Owner

MG2033 commented Nov 23, 2017

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.

@Pelups
Copy link
Author

Pelups commented Nov 23, 2017

Ok ! Thank you for your answer !

@MG2033
Copy link
Owner

MG2033 commented Nov 23, 2017

You're welcome. Also, note that the group convolution operator hasn't been implemented yet officially in Tensorflow.

@MG2033
Copy link
Owner

MG2033 commented Dec 31, 2017

@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.

@YongKun-Liu
Copy link

@MG2033 Would you share the "weights.pkl"

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