[GitHub] [singa] nudles commented on issue #580: [WIP] An experiment for buffering operations

[GitBox <gi...@apache.org>]

nudles commented on issue #580: [WIP] An experiment for buffering operations
URL: https://github.com/apache/singa/pull/580#issuecomment-579577761
 
 
   1. It should be better to return a Tensor instance. To keep it compatible, we can add a new function, sum_at() (returns a tensor), and mark the original sum() as deprecated. When we release v3.0, we mark sum_at as deprecated and change the sum() to return a tensor. This is to follow the convention of the X.Y.Z version style.
   2. Is it more common to pass axis as an integer than an array? Do we need to do summation along multiple dimensions? 

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services