torchstat icon indicating copy to clipboard operation
torchstat copied to clipboard

‘Total MAdd’ and 'Total Flops' results should exchange with each other

Open Lukzin opened this issue 5 years ago • 12 comments

MAdd means that the multiply and add operations are counted as 1 operation while FLOPs count them as 2. Should the results of 'Total MAdd' and 'Total Flops' exchange with each other?

Lukzin avatar May 29 '19 02:05 Lukzin

I have the same question. And why compute_flops need to multiply batch_size but compuute_MAdd didn't. I think there is no need to multiply batch_size, because we only consider one image situation. Is that right?

andyhahaha avatar Jun 04 '19 07:06 andyhahaha

same question

Sixkplus avatar Jul 21 '19 04:07 Sixkplus

same question

DocF avatar Oct 14 '19 08:10 DocF

+1

clamcy avatar Nov 25 '19 11:11 clamcy

+1

supratikbanerjee avatar Jan 28 '20 23:01 supratikbanerjee

+1

qiuhaining avatar Feb 12 '20 08:02 qiuhaining

+1

schutera avatar Mar 11 '20 13:03 schutera

+1

minjs avatar May 08 '20 00:05 minjs

Same question. I reimplement the SwiftNetRN-18 with the MXNet framework, resulting in over 200 GFLOPs of input size 1024*2048 that is similar to (May be) BUGs in paper: wrong Gflops #14

BebDong avatar Jul 23 '20 08:07 BebDong

+1!!!! I think ‘Total MAdd’ and 'Total Flops' results should exchange with each other.

zhoumenghan avatar Oct 18 '21 07:10 zhoumenghan

Same question

hychiang-git avatar Jan 22 '22 17:01 hychiang-git

收到了!谢谢!

qiuhaining avatar Jan 22 '22 17:01 qiuhaining