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

Posted to commits@tvm.apache.org by GitBox <gi...@apache.org> on 2021/04/22 18:12:53 UTC

[GitHub] [tvm] tkonolige commented on pull request #7903: [Graph Executor Debugger] Fix parameter dump

tkonolige commented on pull request #7903:
URL: https://github.com/apache/tvm/pull/7903#issuecomment-825075667


   I agree with removing `debug_get_output`. It is essentially useless.
   
   I can see two use-cases for getting output: you want output from a specific layer or you want the output from each layer. We can implement the latter by repeatedly running the former, but this may be inefficient. An incremental approach still has benefits when running over rpc or on an embedded system.
   
   I think we should use this incremental approach, but I would like the interface to get all outputs to also be available from c++. i.e. can we put as little of the interface in python as possible?
   


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