Re: [apache/tvm-rfcs] Add Project API RFC (#8)

2021-08-04 Thread Tianqi Chen
Merged #8 into main. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/apache/tvm-rfcs/pull/8#event-5111983679

Re: [apache/tvm-rfcs] Add Project API RFC (#8)

2021-07-29 Thread Tianqi Chen
@Mousius @mehrdadh @tkonolige @guberti please take another look and https://tvm.apache.org/docs/contribute/code_review.html#approve-and-request-changes-explicitly -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://githu

Re: [apache/tvm-rfcs] Add Project API RFC (#8)

2021-07-29 Thread Andrew Reusch
@Mousius @tqchen please take a look -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/apache/tvm-rfcs/pull/8#issuecomment-889484620

Re: [apache/tvm-rfcs] Add Project API RFC (#8)

2021-07-27 Thread Andrew Reusch
@tkonolige @guberti @mehrdadh please take another look and explicitly approve if you're good w/ this -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/apache/tvm-rfcs/pull/8#issuecomment-887894876

Re: [apache/tvm-rfcs] Add Project API RFC (#8)

2021-07-27 Thread Andrew Reusch
@Mousius please take another look when you can and explicitly approve! -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/apache/tvm-rfcs/pull/8#issuecomment-887879286

Re: [apache/tvm-rfcs] Add Project API RFC (#8)

2021-07-27 Thread Gustavo Romero
>> On generate_project method, and considering TVMC, my only comment is that it >> should allow a project creation based also on MLF .tar, instead of only a >> "live" executor. >I agree; let's merge additional logic in project.py to do this as follow-on to >this impl. Sure! I'll submit a follow

Re: [apache/tvm-rfcs] Add Project API RFC (#8)

2021-07-26 Thread Andrew Reusch
Thanks @Mousius @gromero for your comments. updated the RFC to reflect accurately on PoC now that it seems to be passing regression. Please take another look so we can merge this and proceed forward with Project API. @gromero some follow-ups on your comments: > On generate_project method, and co

Re: [apache/tvm-rfcs] Add Project API RFC (#8)

2021-06-30 Thread Andrew Reusch
@mdw-octoml would be great to get your feedback here as well -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/apache/tvm-rfcs/pull/8#issuecomment-871810453