leandron commented on pull request #8645: URL: https://github.com/apache/tvm/pull/8645#issuecomment-898336275
@jiangjiajun thanks for the comprehensive answer. A minor comment is that on the `tvmc` line I foresee we would need to point the input format, that's basically part of the frontend implementation. It would be something like this: > ``` > tvmc compile \ > --target "llvm" \ > --model-format paddle \ > --output resnet50-v2-7-tvm.tar \ > dirname/model_name > ``` > How do you think about this solution? And is it better that I put the `tvmc` frontend implementation in this pull request, or send another pull request? I think a separate pull request is a better option, to keep it simpler to review. -- 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]
