GEMM results and gemm_configs are out of sync

baidu-research / DeepBench

Benchmarking Deep Learning operations on different hardware

Apache License 2.0

1.07k stars 239 forks source link

GEMM results and gemm_configs are out of sync #68

Closed dagamayank closed 6 years ago

dagamayank commented 7 years ago

One example:

This spreadsheet has data for the config below: 6144 | 48000 | 2048 | T | N

But the above config is not found in the training_set here: https://github.com/baidu-research/DeepBench/blob/master/code/kernels/gemm_problems.h

sharannarang commented 6 years ago

@dagamayank , Thanks for pointing this out. I have created a PR with the fix. All the kernels with M=6144 were impacted. Could you please take a look and confirm that it addresses this issue?

sharannarang commented 6 years ago

I've merged the PR with the fix. Please let me know if you still see any issues. Thanks!