[GitHub] [incubator-tvm] comaniac commented on a change in pull request #6657: [AutoScheduler] Improve test cases

2020-10-09 Thread GitBox
comaniac commented on a change in pull request #6657: URL: https://github.com/apache/incubator-tvm/pull/6657#discussion_r502756049 ## File path: python/tvm/auto_scheduler/measure_record.py ## @@ -159,3 +175,38 @@ def load_best(filename, workload_key=None, target=None):

[GitHub] [incubator-tvm] lixiaoquan commented on pull request #6658: Revert "[Relay] Keep fixed dim when unifying dynamic shape (#5795)"

2020-10-09 Thread GitBox
lixiaoquan commented on pull request #6658: URL: https://github.com/apache/incubator-tvm/pull/6658#issuecomment-706498484 LGTM. Haven't thought it may introduce issue in some case. This is an automated message from the Apache

[GitHub] [incubator-tvm] merrymercy commented on a change in pull request #6657: [AutoScheduler] Improve test cases

2020-10-09 Thread GitBox
merrymercy commented on a change in pull request #6657: URL: https://github.com/apache/incubator-tvm/pull/6657#discussion_r502754246 ## File path: tests/python/unittest/test_auto_scheduler_measure.py ## @@ -167,53 +167,76 @@ def test_record_pragma_storage_align_rfactor():

[GitHub] [incubator-tvm] merrymercy commented on a change in pull request #6657: [AutoScheduler] Improve test cases

2020-10-09 Thread GitBox
merrymercy commented on a change in pull request #6657: URL: https://github.com/apache/incubator-tvm/pull/6657#discussion_r502754215 ## File path: python/tvm/auto_scheduler/measure_record.py ## @@ -159,3 +175,38 @@ def load_best(filename, workload_key=None, target=None):

[incubator-tvm] branch ci-docker-staging updated (4b7aa5d -> 074dbef)

2020-10-09 Thread jroesch
This is an automated email from the ASF dual-hosted git repository. jroesch pushed a change to branch ci-docker-staging in repository https://gitbox.apache.org/repos/asf/incubator-tvm.git. discard 4b7aa5d Setup Rust path discard 7836654 Add deps needed for Rust examples and docs discard 2e5

[incubator-tvm] branch ci-docker-staging updated (4b7aa5d -> 074dbef)

2020-10-09 Thread jroesch
This is an automated email from the ASF dual-hosted git repository. jroesch pushed a change to branch ci-docker-staging in repository https://gitbox.apache.org/repos/asf/incubator-tvm.git. discard 4b7aa5d Setup Rust path discard 7836654 Add deps needed for Rust examples and docs discard 2e5

[GitHub] [incubator-tvm] comaniac commented on a change in pull request #6657: [AutoScheduler] Improve test cases

2020-10-09 Thread GitBox
comaniac commented on a change in pull request #6657: URL: https://github.com/apache/incubator-tvm/pull/6657#discussion_r502749481 ## File path: python/tvm/auto_scheduler/measure_record.py ## @@ -159,3 +175,38 @@ def load_best(filename, workload_key=None, target=None):

[GitHub] [incubator-tvm] merrymercy commented on a change in pull request #6657: [AutoScheduler] Improve test cases

2020-10-09 Thread GitBox
merrymercy commented on a change in pull request #6657: URL: https://github.com/apache/incubator-tvm/pull/6657#discussion_r502745970 ## File path: python/tvm/auto_scheduler/measure_record.py ## @@ -70,6 +72,13 @@ def read_lines(self, max_lines=None, skip_lines=0):

[GitHub] [incubator-tvm] masahi edited a comment on issue #6594: [Torch, CI] Support PyTorch 1.6

2020-10-09 Thread GitBox
masahi edited a comment on issue #6594: URL: https://github.com/apache/incubator-tvm/issues/6594#issuecomment-706484241 I've sent the revert to https://github.com/apache/incubator-tvm/pull/6658 With this and other minor fixes, I can finally run torchvision detection models from v1.6.

[GitHub] [incubator-tvm] masahi opened a new pull request #6659: [Torch] Object detection support update for PyTorch 1.6

2020-10-09 Thread GitBox
masahi opened a new pull request #6659: URL: https://github.com/apache/incubator-tvm/pull/6659 Depends on https://github.com/apache/incubator-tvm/pull/6658 cc @kevinthesun @yongwww @t-vi @zhiics This is an automated m

[GitHub] [incubator-tvm] masahi commented on issue #6594: [Torch, CI] Support PyTorch 1.6

2020-10-09 Thread GitBox
masahi commented on issue #6594: URL: https://github.com/apache/incubator-tvm/issues/6594#issuecomment-706484241 I've sent the revert to https://github.com/apache/incubator-tvm/pull/6658 With this and other minor fixes, I can finally run torchvision detection models from v1.6 -

[GitHub] [incubator-tvm] masahi opened a new pull request #6658: Revert "[Relay] Keep fixed dim when unifying dynamic shape (#5795)"

2020-10-09 Thread GitBox
masahi opened a new pull request #6658: URL: https://github.com/apache/incubator-tvm/pull/6658 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abov

[GitHub] [incubator-tvm] kevinthesun edited a comment on issue #6594: [Torch, CI] Support PyTorch 1.6

2020-10-09 Thread GitBox
kevinthesun edited a comment on issue #6594: URL: https://github.com/apache/incubator-tvm/issues/6594#issuecomment-706474453 Hmmm, I think that PR is an optimization for some cases but is actually not correct since static dim is just a case of ```Any``` but not vise versa. The case provide

[GitHub] [incubator-tvm] kevinthesun edited a comment on issue #6594: [Torch, CI] Support PyTorch 1.6

2020-10-09 Thread GitBox
kevinthesun edited a comment on issue #6594: URL: https://github.com/apache/incubator-tvm/issues/6594#issuecomment-706474453 Hmmm, I think that PR is an optimization for some cases but is actually not correct since static dim is just a case of ```Any``` but not vise versa. The case provide

[GitHub] [incubator-tvm] merrymercy commented on pull request #6568: [AutoScheduler] Improve the rule of mutating parallel granularity

2020-10-09 Thread GitBox
merrymercy commented on pull request #6568: URL: https://github.com/apache/incubator-tvm/pull/6568#issuecomment-706475273 Fixed by #6657 This is an automated message from the Apache Git Service. To respond to the message, pl

[GitHub] [incubator-tvm] kevinthesun edited a comment on issue #6594: [Torch, CI] Support PyTorch 1.6

2020-10-09 Thread GitBox
kevinthesun edited a comment on issue #6594: URL: https://github.com/apache/incubator-tvm/issues/6594#issuecomment-706474453 Hmmm, I think that PR is an optimization for some cases but is actually not correct since static dim is just a case of ```Any``` but not vise versa. The case provide

[GitHub] [incubator-tvm] kevinthesun edited a comment on issue #6594: [Torch, CI] Support PyTorch 1.6

2020-10-09 Thread GitBox
kevinthesun edited a comment on issue #6594: URL: https://github.com/apache/incubator-tvm/issues/6594#issuecomment-706474453 Hmmm, I think that PR is an optimization for some cases but is actually not correct since static dim is just a case of ```Any``` but not vise versa. @lixiaoquan We s

[GitHub] [incubator-tvm] merrymercy commented on pull request #6657: [AutoScheduler] Improve test cases

2020-10-09 Thread GitBox
merrymercy commented on pull request #6657: URL: https://github.com/apache/incubator-tvm/pull/6657#issuecomment-706474600 cc @jcf94 @comaniac @FrozenGene This is an automated message from the Apache Git Service. To respond t

[GitHub] [incubator-tvm] kevinthesun edited a comment on issue #6594: [Torch, CI] Support PyTorch 1.6

2020-10-09 Thread GitBox
kevinthesun edited a comment on issue #6594: URL: https://github.com/apache/incubator-tvm/issues/6594#issuecomment-706474453 Hmmm, I think that PR is an optimization for some cases but is actually not correct since static dim is just a case of ```Any```. @lixiaoquan We should revert it?

[GitHub] [incubator-tvm] kevinthesun commented on issue #6594: [Torch, CI] Support PyTorch 1.6

2020-10-09 Thread GitBox
kevinthesun commented on issue #6594: URL: https://github.com/apache/incubator-tvm/issues/6594#issuecomment-706474453 Hmmm, I think that PR is an optimization for some cases but is actually not correct. @lixiaoquan We should revert it? -

[GitHub] [incubator-tvm] masahi edited a comment on issue #6594: [Torch, CI] Support PyTorch 1.6

2020-10-09 Thread GitBox
masahi edited a comment on issue #6594: URL: https://github.com/apache/incubator-tvm/issues/6594#issuecomment-706472689 @kevinthesun @jroesch @lixiaoquan @MarisaKirisame I found that this change was introduced in https://github.com/apache/incubator-tvm/pull/5795 If I make that above

[GitHub] [incubator-tvm] masahi edited a comment on issue #6594: [Torch, CI] Support PyTorch 1.6

2020-10-09 Thread GitBox
masahi edited a comment on issue #6594: URL: https://github.com/apache/incubator-tvm/issues/6594#issuecomment-706472689 @kevinthesun @jroesch @lixiaoquan @MarisaKirisame I found that this change was introduced in https://github.com/apache/incubator-tvm/pull/5795 If I make that above

[GitHub] [incubator-tvm] merrymercy opened a new pull request #6657: [AutoScheduler] Improve test cases

2020-10-09 Thread GitBox
merrymercy opened a new pull request #6657: URL: https://github.com/apache/incubator-tvm/pull/6657 - Simplify test cases with the new API (`auto_scheduler.create_task`) - Fix the flaky test (https://github.com/apache/incubator-tvm/pull/6572). The unit test has been tested with 1,000 runs

[GitHub] [incubator-tvm] masahi edited a comment on issue #6594: [Torch, CI] Support PyTorch 1.6

2020-10-09 Thread GitBox
masahi edited a comment on issue #6594: URL: https://github.com/apache/incubator-tvm/issues/6594#issuecomment-706472689 @kevinthesun @jroesch @lixiaoquan @MarisaKirisame I found that this change was introduced in https://github.com/apache/incubator-tvm/pull/5795 If I make that above

[GitHub] [incubator-tvm] masahi edited a comment on issue #6594: [Torch, CI] Support PyTorch 1.6

2020-10-09 Thread GitBox
masahi edited a comment on issue #6594: URL: https://github.com/apache/incubator-tvm/issues/6594#issuecomment-706472689 @kevinthesun @jroesch @lixiaoquan @MarisaKirisame I found that this change was introduced in https://github.com/apache/incubator-tvm/pull/5795 If I make that above

[GitHub] [incubator-tvm] masahi edited a comment on issue #6594: [Torch, CI] Support PyTorch 1.6

2020-10-09 Thread GitBox
masahi edited a comment on issue #6594: URL: https://github.com/apache/incubator-tvm/issues/6594#issuecomment-706472689 @kevinthesun @jroesch @lixiaoquan @MarisaKirisame I found that this change was introduced in https://github.com/apache/incubator-tvm/pull/5795 If I make that above

[GitHub] [incubator-tvm] masahi commented on issue #6594: [Torch, CI] Support PyTorch 1.6

2020-10-09 Thread GitBox
masahi commented on issue #6594: URL: https://github.com/apache/incubator-tvm/issues/6594#issuecomment-706472689 @kevinthesun @jroesch @lixiaoquan I found that this change was introduced in https://github.com/apache/incubator-tvm/pull/5795 If I make that above change, that effectivel

[GitHub] [incubator-tvm] masahi commented on issue #6594: [Torch, CI] Support PyTorch 1.6

2020-10-09 Thread GitBox
masahi commented on issue #6594: URL: https://github.com/apache/incubator-tvm/issues/6594#issuecomment-706471804 Finding suspicious code was easy, here, when lhs is `IntImmNode` and rhs is `AnyNode`, `ulhs` is returned. https://github.com/apache/incubator-tvm/blob/98c2096f4944bdbdbb

[GitHub] [incubator-tvm] masahi commented on issue #6594: [Torch, CI] Support PyTorch 1.6

2020-10-09 Thread GitBox
masahi commented on issue #6594: URL: https://github.com/apache/incubator-tvm/issues/6594#issuecomment-706469099 @kevinthesun thanks for pointing this out and providing a minimum test case. I'm going to take a look at what's going on inside type checker and hopefully fix it. With this fixe

[GitHub] [incubator-tvm] kevinthesun edited a comment on issue #6594: [Torch, CI] Support PyTorch 1.6

2020-10-09 Thread GitBox
kevinthesun edited a comment on issue #6594: URL: https://github.com/apache/incubator-tvm/issues/6594#issuecomment-706467574 @masahi ``` import tvm from tvm import relay dtype = "float32" branch_a = relay.var("a", shape=(relay.Any(),), dtype=dtype) branch_b = rel

[GitHub] [incubator-tvm] kevinthesun edited a comment on issue #6594: [Torch, CI] Support PyTorch 1.6

2020-10-09 Thread GitBox
kevinthesun edited a comment on issue #6594: URL: https://github.com/apache/incubator-tvm/issues/6594#issuecomment-706467574 @masahi ``` import tvm from tvm import relay dtype = "float32" branch_a = relay.var("a", shape=(relay.Any(),), dtype=dtype) branch_b = rel

[GitHub] [incubator-tvm] kevinthesun commented on issue #6594: [Torch, CI] Support PyTorch 1.6

2020-10-09 Thread GitBox
kevinthesun commented on issue #6594: URL: https://github.com/apache/incubator-tvm/issues/6594#issuecomment-706467574 @masahi This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [incubator-tvm] kevinthesun commented on issue #6594: [Torch, CI] Support PyTorch 1.6

2020-10-09 Thread GitBox
kevinthesun commented on issue #6594: URL: https://github.com/apache/incubator-tvm/issues/6594#issuecomment-706461452 @masahi That shape problem coming from ```If``` branch looks like a Relay type inference issue to me. Type inference should generate dynamic shape in this case. -

svn commit: r41878 - in /release/incubator/tvm: tvm-v0.6.1/apache-tvm-src-v0.6.1-incubating.tar.gz.sha512 tvm-v0.7.0/apache-tvm-src-v0.7.0-incubating.tar.gz.sha512

2020-10-09 Thread ziheng
Author: ziheng Date: Sat Oct 10 00:36:55 2020 New Revision: 41878 Log: (empty) Modified: release/incubator/tvm/tvm-v0.6.1/apache-tvm-src-v0.6.1-incubating.tar.gz.sha512 release/incubator/tvm/tvm-v0.7.0/apache-tvm-src-v0.7.0-incubating.tar.gz.sha512 Modified: release/incubator/tvm/tvm-

[GitHub] [incubator-tvm] merrymercy edited a comment on issue #6650: Ansor support for multiprocessing spawn

2020-10-09 Thread GitBox
merrymercy edited a comment on issue #6650: URL: https://github.com/apache/incubator-tvm/issues/6650#issuecomment-706448409 The issue is not about RPC. It is about the argument passing in `multiprocessing.Pool.map`. Some TVMObjects cannot be correctly serialized, so we cannot pass them as

[incubator-tvm-site] branch asf-site updated: Build at Fri Oct 9 20:03:48 EDT 2020

2020-10-09 Thread tqchen
This is an automated email from the ASF dual-hosted git repository. tqchen pushed a commit to branch asf-site in repository https://gitbox.apache.org/repos/asf/incubator-tvm-site.git The following commit(s) were added to refs/heads/asf-site by this push: new c19d9f5 Build at Fri Oct 9 20:

[GitHub] [incubator-tvm] merrymercy edited a comment on issue #6650: Ansor support for multiprocessing spawn

2020-10-09 Thread GitBox
merrymercy edited a comment on issue #6650: URL: https://github.com/apache/incubator-tvm/issues/6650#issuecomment-706448409 The issue is not about RPC. It is about the argument passing in `multiprocessing.Pool.map`. Some TVMObjects cannot be correctly serialized, so we cannot pass them as

[incubator-tvm-site] branch asf-site updated: Build at Fri Oct 9 17:01:35 PDT 2020

2020-10-09 Thread tqchen
This is an automated email from the ASF dual-hosted git repository. tqchen pushed a commit to branch asf-site in repository https://gitbox.apache.org/repos/asf/incubator-tvm-site.git The following commit(s) were added to refs/heads/asf-site by this push: new 77df039 Build at Fri Oct 9 17:

[GitHub] [incubator-tvm] merrymercy edited a comment on issue #6650: Ansor support for multiprocessing spawn

2020-10-09 Thread GitBox
merrymercy edited a comment on issue #6650: URL: https://github.com/apache/incubator-tvm/issues/6650#issuecomment-706448409 The issue is not about RPC. It is about the argument passing in `multiprocessing.Pool.map`. Some TVMObjects cannot be correctly serialized, so we cannot pass them as

[GitHub] [incubator-tvm] merrymercy edited a comment on issue #6650: Ansor support for multiprocessing spawn

2020-10-09 Thread GitBox
merrymercy edited a comment on issue #6650: URL: https://github.com/apache/incubator-tvm/issues/6650#issuecomment-706448409 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [incubator-tvm] merrymercy edited a comment on issue #6650: Ansor support for multiprocessing spawn

2020-10-09 Thread GitBox
merrymercy edited a comment on issue #6650: URL: https://github.com/apache/incubator-tvm/issues/6650#issuecomment-706448409 The issue is not about RPC. It is about the argument passing in `multiprocessing.Pool.map`. Some TVMObjects cannot be correctly serialized, so we cannot pass them as

[GitHub] [incubator-tvm] merrymercy edited a comment on issue #6650: Ansor support for multiprocessing spawn

2020-10-09 Thread GitBox
merrymercy edited a comment on issue #6650: URL: https://github.com/apache/incubator-tvm/issues/6650#issuecomment-706448409 The issue is not about RPC. It is about the argument passing in `multiprocessing.Pool.map`. Some TVMObjects cannot be correctly serialized, so we cannot pass them as

[GitHub] [incubator-tvm] merrymercy commented on issue #6650: Ansor support for multiprocessing spawn

2020-10-09 Thread GitBox
merrymercy commented on issue #6650: URL: https://github.com/apache/incubator-tvm/issues/6650#issuecomment-706448409 The issue is not about RPC. It is about the argument passing in `multiprocessing.Pool.map`. Some TVMObjects cannot be correctly serialized, so we cannot pass them as argumen

[incubator-tvm-site] branch master updated: Update TVM v0.7.0 Release (#18)

2020-10-09 Thread ziheng
This is an automated email from the ASF dual-hosted git repository. ziheng pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-tvm-site.git The following commit(s) were added to refs/heads/master by this push: new 77876e9 Update TVM v0.7.0 Release (

[incubator-tvm-site] 01/01: Update.

2020-10-09 Thread ziheng
This is an automated email from the ASF dual-hosted git repository. ziheng pushed a commit to branch update in repository https://gitbox.apache.org/repos/asf/incubator-tvm-site.git commit 73d0c2aae067cab82ca5bbead4ef63bdcb79dfe3 Author: ZihengJiang AuthorDate: Fri Oct 9 16:38:20 2020 -0700

[incubator-tvm-site] branch update created (now 73d0c2a)

2020-10-09 Thread ziheng
This is an automated email from the ASF dual-hosted git repository. ziheng pushed a change to branch update in repository https://gitbox.apache.org/repos/asf/incubator-tvm-site.git. at 73d0c2a Update. This branch includes the following new commits: new 73d0c2a Update. The 1 revis

svn commit: r41877 - in /release/incubator/tvm/tvm-v0.7.0: apache-tvm-src-v0.7.0-incubating.tar.gz.sha512 apache-tvm-src-v0.7.0.rc0-incubating.tar.gz.sha512

2020-10-09 Thread ziheng
Author: ziheng Date: Fri Oct 9 23:36:39 2020 New Revision: 41877 Log: (empty) Added: release/incubator/tvm/tvm-v0.7.0/apache-tvm-src-v0.7.0-incubating.tar.gz.sha512 - copied unchanged from r41876, release/incubator/tvm/tvm-v0.7.0/apache-tvm-src-v0.7.0.rc0-incubating.tar.gz.sha512 Rem

svn commit: r41876 - in /release/incubator/tvm/tvm-v0.7.0: apache-tvm-src-v0.7.0-incubating.tar.gz.asc apache-tvm-src-v0.7.0.rc0-incubating.tar.gz.asc

2020-10-09 Thread ziheng
Author: ziheng Date: Fri Oct 9 23:36:14 2020 New Revision: 41876 Log: (empty) Added: release/incubator/tvm/tvm-v0.7.0/apache-tvm-src-v0.7.0-incubating.tar.gz.asc - copied unchanged from r41875, release/incubator/tvm/tvm-v0.7.0/apache-tvm-src-v0.7.0.rc0-incubating.tar.gz.asc Removed:

svn commit: r41875 - in /release/incubator/tvm/tvm-v0.7.0: apache-tvm-src-v0.7.0-incubating.tar.gz apache-tvm-src-v0.7.0.rc0-incubating.tar.gz

2020-10-09 Thread ziheng
Author: ziheng Date: Fri Oct 9 23:35:54 2020 New Revision: 41875 Log: (empty) Added: release/incubator/tvm/tvm-v0.7.0/apache-tvm-src-v0.7.0-incubating.tar.gz - copied unchanged from r41874, release/incubator/tvm/tvm-v0.7.0/apache-tvm-src-v0.7.0.rc0-incubating.tar.gz Removed: relea

[incubator-tvm] tag v0.7.0 created (now 728b829)

2020-10-09 Thread ziheng
This is an automated email from the ASF dual-hosted git repository. ziheng pushed a change to tag v0.7.0 in repository https://gitbox.apache.org/repos/asf/incubator-tvm.git. at 728b829 (commit) No new revisions were added by this update.

[incubator-tvm] branch master updated: Faster sparse_dense on GPUs (#6580)

2020-10-09 Thread jroesch
This is an automated email from the ASF dual-hosted git repository. jroesch pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-tvm.git The following commit(s) were added to refs/heads/master by this push: new 6d0351a Faster sparse_dense on GPUs (#6

[GitHub] [incubator-tvm] jroesch merged pull request #6580: Faster sparse_dense on GPUs

2020-10-09 Thread GitBox
jroesch merged pull request #6580: URL: https://github.com/apache/incubator-tvm/pull/6580 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

svn commit: r41873 - /dev/incubator/tvm/tvm-v0.7.0-rc0/ /release/incubator/tvm/tvm-v0.7.0/

2020-10-09 Thread ziheng
Author: ziheng Date: Fri Oct 9 23:13:22 2020 New Revision: 41873 Log: (empty) Added: release/incubator/tvm/tvm-v0.7.0/ - copied from r41872, dev/incubator/tvm/tvm-v0.7.0-rc0/ Removed: dev/incubator/tvm/tvm-v0.7.0-rc0/

[GitHub] [incubator-tvm] jroesch commented on pull request #6656: [Rust][Diagnostics][WIP] Add initial boilerplate for Rust diagnostic interface.

2020-10-09 Thread GitBox
jroesch commented on pull request #6656: URL: https://github.com/apache/incubator-tvm/pull/6656#issuecomment-706418176 cc @gussmith23 and @mwillsey This is an automated message from the Apache Git Service. To respond to the

[GitHub] [incubator-tvm] jroesch opened a new pull request #6656: [Rust][Diagnostics][WIP] Add initial boilerplate for Rust diagnostic interface.

2020-10-09 Thread GitBox
jroesch opened a new pull request #6656: URL: https://github.com/apache/incubator-tvm/pull/6656 This PR exposes new diagnostics to Rust so we can reuse the better libraries for diagnostics rendering via Rust. This is an aut

[GitHub] [incubator-tvm] d-smirnov commented on a change in pull request #6532: [BYOC][ACL] Support add operation

2020-10-09 Thread GitBox
d-smirnov commented on a change in pull request #6532: URL: https://github.com/apache/incubator-tvm/pull/6532#discussion_r502667277 ## File path: tests/python/contrib/test_arm_compute_lib/test_add.py ## @@ -0,0 +1,135 @@ +# Licensed to the Apache Software Foundation (ASF) under

[GitHub] [incubator-tvm] codeislife99 commented on pull request #6641: Modify annotate_target

2020-10-09 Thread GitBox
codeislife99 commented on pull request #6641: URL: https://github.com/apache/incubator-tvm/pull/6641#issuecomment-706389962 @leandron @junrushao1994 The PR is now ready for review. Please take a look. @comaniac @zhiics T

[GitHub] [incubator-tvm] codeislife99 edited a comment on pull request #6641: Modify annotate_target

2020-10-09 Thread GitBox
codeislife99 edited a comment on pull request #6641: URL: https://github.com/apache/incubator-tvm/pull/6641#issuecomment-706389962 @leandron @junrushao1994 The PR is now ready for review. Please take a look. cc: @comaniac @zhiics ---

[GitHub] [incubator-tvm] comaniac commented on pull request #6655: [BYOC] Added default_tuples parameter to AnnotateTarget pass

2020-10-09 Thread GitBox
comaniac commented on pull request #6655: URL: https://github.com/apache/incubator-tvm/pull/6655#issuecomment-706369034 Thanks for the PR. I have a hard deadline this weekend and will review it after next Tuesday. This is an

[GitHub] [incubator-tvm] leandron commented on issue #6642: [Test] patterns for check into tests/python/contrib/test_tedd.py mismatch when running on CPU

2020-10-09 Thread GitBox
leandron commented on issue #6642: URL: https://github.com/apache/incubator-tvm/issues/6642#issuecomment-706362371 I consider this is closed now This is an automated message from the Apache Git Service. To respond to the mess

[GitHub] [incubator-tvm] leandron closed issue #6642: [Test] patterns for check into tests/python/contrib/test_tedd.py mismatch when running on CPU

2020-10-09 Thread GitBox
leandron closed issue #6642: URL: https://github.com/apache/incubator-tvm/issues/6642 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 t

[GitHub] [incubator-tvm] d-smirnov opened a new pull request #6655: [BYOC] Added default_tuples parameter to AnnotateTarget pass

2020-10-09 Thread GitBox
d-smirnov opened a new pull request #6655: URL: https://github.com/apache/incubator-tvm/pull/6655 This PR adds "default_tuples" parameter to AnnotateTarget pass. When set (default_tuples=True) it prevents tuples to be promoted to previously annotated operations. This is useful in case if y

[GitHub] [incubator-tvm] tmoreau89 commented on pull request #6274: [Diagnostics][Relay][InferType] Refactor InferType to work on whole module, and use new diagnostics.

2020-10-09 Thread GitBox
tmoreau89 commented on pull request #6274: URL: https://github.com/apache/incubator-tvm/pull/6274#issuecomment-706344558 Thank you @jroesch @mbrookhart @rkimball the PR has been merged. This is an automated message from the A

[GitHub] [incubator-tvm] tmoreau89 merged pull request #6274: [Diagnostics][Relay][InferType] Refactor InferType to work on whole module, and use new diagnostics.

2020-10-09 Thread GitBox
tmoreau89 merged pull request #6274: URL: https://github.com/apache/incubator-tvm/pull/6274 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 t

[GitHub] [incubator-tvm] areusch commented on issue #6653: Incorrect data type in relay causes abort() rather than exception

2020-10-09 Thread GitBox
areusch commented on issue #6653: URL: https://github.com/apache/incubator-tvm/issues/6653#issuecomment-706342580 it's probably aborting because an exception was raised while unwinding the stack because of a different exception? I don't think you need to actually raise the exception

[GitHub] [incubator-tvm] tqchen commented on issue #6653: Incorrect data type in relay causes abort() rather than exception

2020-10-09 Thread GitBox
tqchen commented on issue #6653: URL: https://github.com/apache/incubator-tvm/issues/6653#issuecomment-706340039 I agree that destructor throw should be rare(we only have very few of them), but in this particular case it is intended behavior(in the AttrInit). And we need to use this way to

[GitHub] [incubator-tvm] tqchen commented on issue #6650: Ansor support for multiprocessing spawn

2020-10-09 Thread GitBox
tqchen commented on issue #6650: URL: https://github.com/apache/incubator-tvm/issues/6650#issuecomment-706319603 There are two factors here, the cost of build and cost of benchmarking. In terms of benchmarking, we do not need multi-processing and can multiplex things through RPC, e.g. try

[GitHub] [incubator-tvm] tkonolige edited a comment on issue #6650: Ansor support for multiprocessing spawn

2020-10-09 Thread GitBox
tkonolige edited a comment on issue #6650: URL: https://github.com/apache/incubator-tvm/issues/6650#issuecomment-706295726 I'm also seeing a similar issue with `python3 -m tvm.driver.tvmc tune --target llvm --output autotuner_records.json resnet50-v2-7.onnx`. It looks like something in aut

[GitHub] [incubator-tvm] tkonolige commented on issue #6650: Ansor support for multiprocessing spawn

2020-10-09 Thread GitBox
tkonolige commented on issue #6650: URL: https://github.com/apache/incubator-tvm/issues/6650#issuecomment-706295726 I'm also seeing a similar issue with `python3 -m tvm.driver.tvmc tune --target llvm --output autotuner_records.json ~/Downloads/resnet50-v2-7.onnx`. It looks like something i

[GitHub] [incubator-tvm] leandron commented on a change in pull request #6654: [Docker][CI][BYODT] add universal to Docker image

2020-10-09 Thread GitBox
leandron commented on a change in pull request #6654: URL: https://github.com/apache/incubator-tvm/pull/6654#discussion_r502549614 ## File path: docker/install/ubuntu_install_universal.sh ## @@ -0,0 +1,26 @@ +#!/bin/bash +# Licensed to the Apache Software Foundation (ASF) under

[GitHub] [incubator-tvm] areusch commented on issue #6653: Incorrect data type in relay causes abort() rather than exception

2020-10-09 Thread GitBox
areusch commented on issue #6653: URL: https://github.com/apache/incubator-tvm/issues/6653#issuecomment-706276834 My POV is that throwing in a destructor is bad because it will necessarily obscure any LOG(FATAL) raised during the lifetime of the object. This could happen from LOG(FATAL) in

[GitHub] [incubator-tvm] tkonolige edited a comment on issue #6650: Ansor support for multiprocessing spawn

2020-10-09 Thread GitBox
tkonolige edited a comment on issue #6650: URL: https://github.com/apache/incubator-tvm/issues/6650#issuecomment-706271838 These lines are the issue: https://github.com/apache/incubator-tvm/blob/master/python/tvm/auto_scheduler/measure.py#L63-L66. The globals are not propagated to the chil

[GitHub] [incubator-tvm] hypercubestart commented on a change in pull request #6654: [Docker][CI][BYODT] add universal to Docker image

2020-10-09 Thread GitBox
hypercubestart commented on a change in pull request #6654: URL: https://github.com/apache/incubator-tvm/pull/6654#discussion_r502535745 ## File path: docker/install/ubuntu_install_universal.sh ## @@ -0,0 +1,26 @@ +#!/bin/bash +# Licensed to the Apache Software Foundation (ASF)

[GitHub] [incubator-tvm] tkonolige commented on issue #6650: Ansor support for multiprocessing spawn

2020-10-09 Thread GitBox
tkonolige commented on issue #6650: URL: https://github.com/apache/incubator-tvm/issues/6650#issuecomment-706271838 These lines are the issue: https://github.com/apache/incubator-tvm/blob/master/python/tvm/auto_scheduler/measure.py#L63-L66. The globals are not propagated to the child proce

[incubator-tvm] branch master updated (0922d17 -> f73a1f6)

2020-10-09 Thread tqchen
This is an automated email from the ASF dual-hosted git repository. tqchen pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-tvm.git. from 0922d17 [FIX,AUTOTVM] More descriptive error message when an autotvm task is not (#6652) add f73a1f6 [

[GitHub] [incubator-tvm] tqchen merged pull request #6643: [TEST][TEDD] improve TEDD tests to also run on CPU Docker image

2020-10-09 Thread GitBox
tqchen merged pull request #6643: URL: https://github.com/apache/incubator-tvm/pull/6643 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 g

[GitHub] [incubator-tvm] hypercubestart commented on a change in pull request #6654: [Docker][CI][BYODT] add universal to Docker image

2020-10-09 Thread GitBox
hypercubestart commented on a change in pull request #6654: URL: https://github.com/apache/incubator-tvm/pull/6654#discussion_r502534589 ## File path: docker/install/ubuntu_install_universal.sh ## @@ -0,0 +1,26 @@ +#!/bin/bash +# Licensed to the Apache Software Foundation (ASF)

[GitHub] [incubator-tvm] tkonolige commented on a change in pull request #6533: Scatter on Cuda

2020-10-09 Thread GitBox
tkonolige commented on a change in pull request #6533: URL: https://github.com/apache/incubator-tvm/pull/6533#discussion_r502532481 ## File path: python/tvm/topi/cuda/scatter.py ## @@ -0,0 +1,444 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contri

[GitHub] [incubator-tvm] mbrookhart commented on a change in pull request #6533: Scatter on Cuda

2020-10-09 Thread GitBox
mbrookhart commented on a change in pull request #6533: URL: https://github.com/apache/incubator-tvm/pull/6533#discussion_r502485425 ## File path: python/tvm/topi/cuda/scatter.py ## @@ -0,0 +1,444 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contr

[GitHub] [incubator-tvm] leandron commented on pull request #6643: [TEST][TEDD] improve TEDD tests to also run on CPU Docker image

2020-10-09 Thread GitBox
leandron commented on pull request #6643: URL: https://github.com/apache/incubator-tvm/pull/6643#issuecomment-706214075 If this possible to merge this? Just asking, so that I can fix my own CI. This is an automated message fr

[GitHub] [incubator-tvm] leandron edited a comment on pull request #6643: [TEST][TEDD] improve TEDD tests to also run on CPU Docker image

2020-10-09 Thread GitBox
leandron edited a comment on pull request #6643: URL: https://github.com/apache/incubator-tvm/pull/6643#issuecomment-706214075 If possible, can we merge this? Just asking, so that I can fix my own CI. This is an automated mes

[GitHub] [incubator-tvm] tqchen commented on issue #6653: Incorrect data type in relay causes abort() rather than exception

2020-10-09 Thread GitBox
tqchen commented on issue #6653: URL: https://github.com/apache/incubator-tvm/issues/6653#issuecomment-706206178 In this case the problem is not in the throw, but the exception being raised in a destructor. The particular destructor is already being marked as enable throw and is fine, so i

[GitHub] [incubator-tvm] tqchen commented on issue #6650: Ansor support for multiprocessing spawn

2020-10-09 Thread GitBox
tqchen commented on issue #6650: URL: https://github.com/apache/incubator-tvm/issues/6650#issuecomment-706208023 Would be great to confirm if this is caused by RPC itself being created via multi-processing and whether starting a separate RPC server would fix the problem. Multiproces

[GitHub] [incubator-tvm] tqchen edited a comment on issue #6653: Incorrect data type in relay causes abort() rather than exception

2020-10-09 Thread GitBox
tqchen edited a comment on issue #6653: URL: https://github.com/apache/incubator-tvm/issues/6653#issuecomment-706206178 In this case the problem is not in the throw, but the exception being raised in a destructor. The particular destructor is already being marked as enable throw and is fin

[GitHub] [incubator-tvm] leandron commented on pull request #6645: [TEST][CI] make sure graphviz is on both ci-cpu and ci-gpu images

2020-10-09 Thread GitBox
leandron commented on pull request #6645: URL: https://github.com/apache/incubator-tvm/pull/6645#issuecomment-705444114 Also cc @zhiics @FrozenGene for reviews and merge if you are happy with this change This is an automated

[GitHub] [incubator-tvm] comaniac commented on pull request #6652: [FIX,AUTOTVM] More descriptive error message when an autotvm task is not found

2020-10-09 Thread GitBox
comaniac commented on pull request #6652: URL: https://github.com/apache/incubator-tvm/pull/6652#issuecomment-705963786 Thanks @tkonolige This is an automated message from the Apache Git Service. To respond to the message, p

[GitHub] [incubator-tvm] zhiics merged pull request #6645: [TEST][CI] make sure graphviz is on both ci-cpu and ci-gpu images

2020-10-09 Thread GitBox
zhiics merged pull request #6645: URL: https://github.com/apache/incubator-tvm/pull/6645 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 g

[GitHub] [incubator-tvm] tqchen commented on issue #6559: [Rust] Memory leak in NDArray

2020-10-09 Thread GitBox
tqchen commented on issue #6559: URL: https://github.com/apache/incubator-tvm/issues/6559#issuecomment-705658977 @jroesch please followup to see if https://github.com/apache/incubator-tvm/pull/6563 fixed the issue This is a

[GitHub] [incubator-tvm] tqchen closed issue #5216: [AutoTVM] RPCRunner network overhead

2020-10-09 Thread GitBox
tqchen closed issue #5216: URL: https://github.com/apache/incubator-tvm/issues/5216 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

[GitHub] [incubator-tvm] liangfu commented on a change in pull request #6603: Add µTVM Zephyr support + QEMU regression test

2020-10-09 Thread GitBox
liangfu commented on a change in pull request #6603: URL: https://github.com/apache/incubator-tvm/pull/6603#discussion_r502392723 ## File path: python/tvm/micro/contrib/zephyr.py ## @@ -0,0 +1,621 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contr

[GitHub] [incubator-tvm] u99127 commented on pull request #6191: quant support for alu-only op

2020-10-09 Thread GitBox
u99127 commented on pull request #6191: URL: https://github.com/apache/incubator-tvm/pull/6191#issuecomment-705814290 Gentle request to put something like [VTA] in your subject line so that a git log shows a useful message.

[GitHub] [incubator-tvm] mbrookhart commented on a change in pull request #6533: Scatter on Cuda

2020-10-09 Thread GitBox
mbrookhart commented on a change in pull request #6533: URL: https://github.com/apache/incubator-tvm/pull/6533#discussion_r501815195 ## File path: tests/python/relay/test_op_level3.py ## @@ -903,8 +904,8 @@ def verify_scatter(dshape, ishape, axis=0): indices_np = np.ra

[GitHub] [incubator-tvm] jwfromm commented on a change in pull request #6647: Add Range op to ONNX, make tvm arange shape_func support negative steps

2020-10-09 Thread GitBox
jwfromm commented on a change in pull request #6647: URL: https://github.com/apache/incubator-tvm/pull/6647#discussion_r501810479 ## File path: python/tvm/relay/op/_transform.py ## @@ -123,7 +123,10 @@ def compute_scatter_add(attrs, inputs, output_type): @script def _arange_s

[GitHub] [incubator-tvm] comaniac merged pull request #6652: [FIX,AUTOTVM] More descriptive error message when an autotvm task is not found

2020-10-09 Thread GitBox
comaniac merged pull request #6652: URL: https://github.com/apache/incubator-tvm/pull/6652 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

[GitHub] [incubator-tvm] tmoreau89 commented on pull request #6647: Add Range op to ONNX, make tvm arange shape_func support negative steps

2020-10-09 Thread GitBox
tmoreau89 commented on pull request #6647: URL: https://github.com/apache/incubator-tvm/pull/6647#issuecomment-705698491 Thanks @mbrookhart @jwfromm the PR has been merged! This is an automated message from the Apache Git Ser

[GitHub] [incubator-tvm] tmoreau89 merged pull request #6647: Add Range op to ONNX, make tvm arange shape_func support negative steps

2020-10-09 Thread GitBox
tmoreau89 merged pull request #6647: URL: https://github.com/apache/incubator-tvm/pull/6647 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 t

[GitHub] [incubator-tvm] zhiics commented on issue #4178: [RFC][VM] Heterogeneous execution in Relay VM

2020-10-09 Thread GitBox
zhiics commented on issue #4178: URL: https://github.com/apache/incubator-tvm/issues/4178#issuecomment-705664905 ahh, thanks for reminding. This is closed by #6337 This is an automated message from the Apache Git Service. To

[GitHub] [incubator-tvm] leandron commented on a change in pull request #6654: [Docker][CI][BYODT] add universal to Docker image

2020-10-09 Thread GitBox
leandron commented on a change in pull request #6654: URL: https://github.com/apache/incubator-tvm/pull/6654#discussion_r502264771 ## File path: docker/install/ubuntu_install_universal.sh ## @@ -0,0 +1,26 @@ +#!/bin/bash +# Licensed to the Apache Software Foundation (ASF) under

[GitHub] [incubator-tvm] tkonolige commented on a change in pull request #6580: Faster sparse_dense on GPUs

2020-10-09 Thread GitBox
tkonolige commented on a change in pull request #6580: URL: https://github.com/apache/incubator-tvm/pull/6580#discussion_r501889549 ## File path: python/tvm/topi/nn/sparse.py ## @@ -207,3 +207,28 @@ def _csr_transpose_ir(data, indices, indptr, out_data, out_indices, out_indptr

[GitHub] [incubator-tvm] tqchen commented on pull request #6651: Fix leakyReLU support for CoreML

2020-10-09 Thread GitBox
tqchen commented on pull request #6651: URL: https://github.com/apache/incubator-tvm/pull/6651#issuecomment-705539490 Thanks @ishitatsuyuki ! This is an automated message from the Apache Git Service. To respond to the message

  1   2   >