[GitHub] [incubator-mxnet] larroy commented on issue #15285: [WIP] Graph dumper

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

larroy commented on issue #15285: [WIP] Graph dumper
URL: https://github.com/apache/incubator-mxnet/pull/15285#issuecomment-507402022
 
 
   @apeforest How can we keep it in a different repo if we have to call it inside backward?  You mean a private branch?
   
   There's indeed some overlap but it's simpler to have a C++ only component that dumps the graph for debug instead of going through non-specific JSon -> having to introduce python. Also that overlap is with NNVM which is an external repo.
   
   I would vote to have this merged which I though twas the original intention and as @anirudh2290 is not blocking it. 
   
   The PR is ready missing fixing a minor CI issue with amalgamation. If it's not going to be merged I will abandon it.
   
   
   

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