jonatan1626 commented on issue #17331: [mxnet 2.0] [item 2.4] Turning on large tensor support by default URL: https://github.com/apache/incubator-mxnet/issues/17331#issuecomment-582752477 @apeforest Oh sorry, so I'm multiplying by only for the samples/second column -1 to keep the meaning consistent with everything else. The rest of the columns depict the correct positive percentage improvement and negative percentage degradation. For example if MKL_LT gives 66 samples/sec and MKL gives 70 samples/sec that will be: 1-(66/70) or 6%. Because it's positive, we think that it's better but actually it's worse because the throughput has gone down. On the other hand if MKL_LT gives 74 samples/sec and MKL gives 70 samples/sec that will be: 1-(74/70) or -5%. Because it's negative, we think it's worse but actually it's better because our throughput has gone up. So I multiply by -1 to give it the same meaning as the rest of the percentages, where positive is better and negative is worse.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
