Closed kalyc closed 6 years ago
Addressed comments. Testing on benchmark model not yet done.
Addressed refactoring comments
Thanks. LGTM. Going ahead with merging these operators. Please follow up with an end to end example using sparse tensor (may be think of having this end to end example work for your benchmarking as well)
Yes will do - we will need to update the existing benchmark script to use these operators as well.
Summary
Add sparse support for
sum
,mean
anddot
operatorsRelated Issues
Continuing - https://github.com/awslabs/keras-apache-mxnet/pull/159 Missing sparse operators
PR Overview