[GitHub] eric-haibin-lin commented on issue #7569: should add row_sparse_push for kvstore

[gi...@git.apache.org]

eric-haibin-lin commented on issue #7569: should add row_sparse_push for kvstore
URL: https://github.com/apache/incubator-mxnet/issues/7569#issuecomment-324210209
 
 
   The push method accepts row_sparse values already, but for pull it requires row id that's why row_sparse_pull was added. 
   
   On 2017?8?22?, at 19:40, Zhang Ming <no...@github.com> wrote:
   
   hi, all,
   I noticed that you have add a function for kvstore , named 'row_sparse_pull', why not to add a 'row_sparse_push'. it is very like my 'partial weight' which is in https://github.com/starimpact/mxnet_v0.8.0/blob/b8ba5a9c8b6d6b7876716a37fbc9284ecf55b920/python/mxnet/kvstore.py#L241
   my partial weights can support distribution training.
   
   ?
   You are receiving this because you are subscribed to this thread.
   Reply to this email directly, view it on GitHub, or mute the thread.
   
   
 
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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