You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@mxnet.apache.org by GitBox <gi...@apache.org> on 2018/03/15 19:23:58 UTC

[GitHub] aaronmarkham commented on issue #10081: [MXNET-82] [WIP] Sparse op tutorial for developers

aaronmarkham commented on issue #10081: [MXNET-82] [WIP] Sparse op tutorial for developers
URL: https://github.com/apache/incubator-mxnet/pull/10081#issuecomment-373494205
 
 
   Prerequisite add op in backend doc updates:
   
   * References a move that seems to have happened, but now the files are in both places... 
   https://github.com/apache/incubator-mxnet/blame/master/docs/faq/add_op_in_backend.md#L48-L51
   
   * Link to specific instructions on adding descriptions
   https://github.com/apache/incubator-mxnet/blame/master/docs/faq/add_op_in_backend.md#L95
   
   * Reference to FCompute is pretty light, and I'm unclear on the comparison with FComputeX in this sparse tutorial.
   
   For my review on the tutorial for sparse ops, it looks like it was outdated before I finished it, so please take a look at all of my "outdated" comments!
   
   

----------------------------------------------------------------
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