benchmarks icon indicating copy to clipboard operation
benchmarks copied to clipboard

DenseNet in this benchmark has "naive" implementaiton, will underperform

Open ahundt opened this issue 7 years ago • 1 comments

At the time of writing, DenseNet is implemented in this benchmark with what is described as the "naive" implementation in Memory-Efficient Implementation of DenseNets. This will under perform compared to the ideal implementation because recursive tf.concat will lead to excessive memory usage.

Details are in the upstream tf issuse https://github.com/tensorflow/tensorflow/issues/12948.

ahundt avatar Dec 13 '17 06:12 ahundt

We currently don't have anybody working on this. It would be great if you could help us by working on this and submitting a PR. Let us know if you need further clarification. Thanks!

bignamehyp avatar Jan 03 '18 08:01 bignamehyp