tvm icon indicating copy to clipboard operation
tvm copied to clipboard

Map batch matmul from PT to TVM.

Open kimishpatel opened this issue 6 years ago • 5 comments

Also added test.

Summary:

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

kimishpatel avatar Aug 20 '19 15:08 kimishpatel

Regarding the error: https://stackoverflow.com/questions/34819221/why-is-python-setup-py-saying-invalid-command-bdist-wheel-on-travis-ci

yinghai avatar Aug 20 '19 20:08 yinghai

Hmm, let me try with updating ci config. Thanks for the link.

From: Yinghai Lu [email protected] Reply-To: pytorch/tvm [email protected] Date: Tuesday, August 20, 2019 at 1:50 PM To: pytorch/tvm [email protected] Cc: Kimish Patel [email protected], Author [email protected] Subject: Re: [pytorch/tvm] Map batch matmul from PT to TVM. (#95)

Regarding the error: https://stackoverflow.com/questions/34819221/why-is-python-setup-py-saying-invalid-command-bdist-wheel-on-travis-cihttps://urldefense.proofpoint.com/v2/url?u=https-3A__stackoverflow.com_questions_34819221_why-2Dis-2Dpython-2Dsetup-2Dpy-2Dsaying-2Dinvalid-2Dcommand-2Dbdist-2Dwheel-2Don-2Dtravis-2Dci&d=DwMCaQ&c=5VD0RTtNlTh3ycd41b3MUw&r=fnncU1KylaEiKyqiohSbhNpassAIOcfsTdnJB1l3C2M&m=5tLmVEBne0DABRuy45PoNFfg9WQAJB1UJFr6cFM-Goc&s=-k_btIbFPZiAwncEeK5r6Q6MA52f3S5n-QhfjxvgB7s&e=

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/pytorch/tvm/pull/95?email_source=notifications&email_token=AEFNYVI356LXONDQAOKZQZTQFRKFNA5CNFSM4INYQBV2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4XTUHI#issuecomment-523188765, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AEFNYVNOW2NHMGVYZUWIDFTQFRKFNANCNFSM4INYQBVQ.

kimishpatel avatar Aug 20 '19 20:08 kimishpatel

Hmm, try rebasing and it should work.

yinghai avatar Aug 20 '19 23:08 yinghai

Rebase to master and we should expect CI to pass.

yinghai avatar Aug 22 '19 00:08 yinghai

This one is causing some correctness issues when run with bert model, even though unit tests pass. So I need to do some investigation before pushing it in.

kimishpatel avatar Aug 22 '19 15:08 kimishpatel