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: [email protected]
