vvchernov opened a new pull request #10293: URL: https://github.com/apache/tvm/pull/10293
Implementation of `set_input_with_index` method in VirtualMachine class. It provides more convinient way to work with multiple input models on native side. I can insert functionality of `set_input_with_index` method to base method `set_input`. But I do not think that it is good idea to mix them. Other possible extension is to additionally reload `set_input` with args: func_name, index1, tensor1, index2, tensor2, …, indexN, tensorN, where N is number of all input tensors. But it is still not perfect solution for me. I think it will be good if we have the same API for both GraphExecutor and VirtualMachine. But I do not see simple way for it. I even do not know how to start motion to this idea. Current solution seems half-hearted for me. @jwfromm @tmoreau89 please see and discuss -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
