[incubator-tvm] branch master updated (d892881 -> 3684a51)

2020-08-03 Thread zhic
This is an automated email from the ASF dual-hosted git repository. zhic pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-tvm.git. from d892881 [BUILD] Remove libtopi from the build (#6189) add 3684a51 [FoldConstant] Create Interpreter for

[GitHub] [incubator-tvm] zhiics merged pull request #6195: [FoldConstant] Create Interpreter for each constant subgraph

2020-08-03 Thread GitBox
zhiics merged pull request #6195: URL: https://github.com/apache/incubator-tvm/pull/6195 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] mbrookhart opened a new pull request #6198: [Relay][Dynamic] Add Dynamic Resize Op

2020-08-03 Thread GitBox
mbrookhart opened a new pull request #6198: URL: https://github.com/apache/incubator-tvm/pull/6198 Thanks to @icemelon9 for some help debugging a unit test. cc @zhiics @electriclilies This is an automated

[GitHub] [incubator-tvm] zhiics commented on a change in pull request #6193: [DOCS] Update pass infra tutorial

2020-08-03 Thread GitBox
zhiics commented on a change in pull request #6193: URL: https://github.com/apache/incubator-tvm/pull/6193#discussion_r464549657 ## File path: docs/dev/use_pass_infra.rst ## @@ -0,0 +1,117 @@ +.. Licensed to the Apache Software Foundation (ASF) under one +or more

[GitHub] [incubator-tvm] zhiics commented on a change in pull request #6193: [DOCS] Update pass infra tutorial

2020-08-03 Thread GitBox
zhiics commented on a change in pull request #6193: URL: https://github.com/apache/incubator-tvm/pull/6193#discussion_r464549657 ## File path: docs/dev/use_pass_infra.rst ## @@ -0,0 +1,117 @@ +.. Licensed to the Apache Software Foundation (ASF) under one +or more

[GitHub] [incubator-tvm] junrushao1994 opened a new pull request #6199: [Target] Rename target_id => target_kind

2020-08-03 Thread GitBox
junrushao1994 opened a new pull request #6199: URL: https://github.com/apache/incubator-tvm/pull/6199 Per [discussion](https://discuss.tvm.ai/t/rfc-tvm-target-specification/6844/44?u=junrushao1994) This is an automated

[GitHub] [incubator-tvm] mbaret commented on pull request #6171: [CI][ETHOSN] Enable CI for Ethos-N

2020-08-03 Thread GitBox
mbaret commented on pull request #6171: URL: https://github.com/apache/incubator-tvm/pull/6171#issuecomment-668088055 ping @tqchen, is there anything else that needs to be done here? This is an automated message from the

[GitHub] [incubator-tvm] zhiics commented on pull request #6124: [Relay] change device annotation from post DFS to recursive

2020-08-03 Thread GitBox
zhiics commented on pull request #6124: URL: https://github.com/apache/incubator-tvm/pull/6124#issuecomment-668093739 please trigger the ci again. This is an automated message from the Apache Git Service. To respond to the

[GitHub] [incubator-tvm] ANSHUMAN87 commented on pull request #6066: [TIR][Transform] HoistIfThenElse added

2020-08-03 Thread GitBox
ANSHUMAN87 commented on pull request #6066: URL: https://github.com/apache/incubator-tvm/pull/6066#issuecomment-668167439 @kevinthesun : Thanks for your response! I am familiar with the scenario you are referring for conv op. I agree to the possibilities of hoisting in that case.

[GitHub] [incubator-tvm] mbrookhart commented on a change in pull request #6124: [Relay] change device annotation from post DFS to recursive

2020-08-03 Thread GitBox
mbrookhart commented on a change in pull request #6124: URL: https://github.com/apache/incubator-tvm/pull/6124#discussion_r464491292 ## File path: tests/python/relay/test_pass_annotation.py ## @@ -309,6 +309,76 @@ def test_visitor_annotation(): test_visitor_annotation()

[GitHub] [incubator-tvm] junrushao1994 commented on pull request #6195: [FoldConstant] Create Interpreter for each constant subgraph

2020-08-03 Thread GitBox
junrushao1994 commented on pull request #6195: URL: https://github.com/apache/incubator-tvm/pull/6195#issuecomment-668127622 It is consistent with your findings CC: @were This is an automated message from the Apache Git

[GitHub] [incubator-tvm] zhiics commented on pull request #6195: [FoldConstant] Create Interpreter for each constant subgraph

2020-08-03 Thread GitBox
zhiics commented on pull request #6195: URL: https://github.com/apache/incubator-tvm/pull/6195#issuecomment-668121375 Thanks @anijain2305 @MarisaKirisame This is an automated message from the Apache Git Service. To respond

[GitHub] [incubator-tvm] tqchen commented on a change in pull request #6193: [DOCS] Update pass infra tutorial

2020-08-03 Thread GitBox
tqchen commented on a change in pull request #6193: URL: https://github.com/apache/incubator-tvm/pull/6193#discussion_r46452 ## File path: docs/dev/use_pass_infra.rst ## @@ -0,0 +1,117 @@ +.. Licensed to the Apache Software Foundation (ASF) under one +or more

[GitHub] [incubator-tvm] electriclilies commented on a change in pull request #6198: [Relay][Dynamic] Add Dynamic Resize Op

2020-08-03 Thread GitBox
electriclilies commented on a change in pull request #6198: URL: https://github.com/apache/incubator-tvm/pull/6198#discussion_r464558226 ## File path: python/tvm/relay/op/image/image.py ## @@ -239,6 +245,7 @@ def affine_grid(data, target_shape=None): """ return

[GitHub] [incubator-tvm] siju-samuel opened a new pull request #6196: [COREML]Unary ops support added in frontend

2020-08-03 Thread GitBox
siju-samuel opened a new pull request #6196: URL: https://github.com/apache/incubator-tvm/pull/6196 @FrozenGene @masahi @yzhliu Please help to review this PR. This is an automated message from the Apache Git

[GitHub] [incubator-tvm] 12101111 opened a new issue #6197: [Rust] TVMError: Check failed: type_code_ == kDLInt (1 vs. 0) : expected int but get uint

2020-08-03 Thread GitBox
1210 opened a new issue #6197: URL: https://github.com/apache/incubator-tvm/issues/6197 I got this when trying example of rust frontend: ``` original image dimensions: (256, 256) resized image dimensions: (224, 224) input size is [1, 3, 224, 224] thread 'main' panicked at

[GitHub] [incubator-tvm] leandron edited a comment on pull request #6167: [TFLite] Implemented PADV2 Operator for TFLite and added support for constant values in PAD.

2020-08-03 Thread GitBox
leandron edited a comment on pull request #6167: URL: https://github.com/apache/incubator-tvm/pull/6167#issuecomment-666226823 cc @anijain2305 @u99127 @FrozenGene @tqchen This is an automated message from the Apache Git

[GitHub] [incubator-tvm] MarisaKirisame commented on pull request #6195: [FoldConstant] Create Interpreter for each constant subgraph

2020-08-03 Thread GitBox
MarisaKirisame commented on pull request #6195: URL: https://github.com/apache/incubator-tvm/pull/6195#issuecomment-667955487 This look right. This is an automated message from the Apache Git Service. To respond to the

[GitHub] [incubator-tvm] FrozenGene commented on a change in pull request #6196: [COREML]Unary ops support added in frontend

2020-08-03 Thread GitBox
FrozenGene commented on a change in pull request #6196: URL: https://github.com/apache/incubator-tvm/pull/6196#discussion_r464416067 ## File path: python/tvm/relay/frontend/coreml.py ## @@ -350,6 +350,33 @@ def _MinLayerParams(op, inexpr, etab): return _min +def

[GitHub] [incubator-tvm] merrymercy merged pull request #6194: [TIR] Enhance VerifyGPUCode

2020-08-03 Thread GitBox
merrymercy merged pull request #6194: URL: https://github.com/apache/incubator-tvm/pull/6194 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

[GitHub] [incubator-tvm] MarisaKirisame commented on pull request #6152: [RELAY] Basic block normal form

2020-08-03 Thread GitBox
MarisaKirisame commented on pull request #6152: URL: https://github.com/apache/incubator-tvm/pull/6152#issuecomment-668232639 Thanks @eric-haibin-lin @mbrookhart @junrushao1994 This is an automated message from the Apache

[GitHub] [incubator-tvm] tqchen commented on issue #6179: Fix conv2d grad for strided cases under the new conv2d_transpose def

2020-08-03 Thread GitBox
tqchen commented on issue #6179: URL: https://github.com/apache/incubator-tvm/issues/6179#issuecomment-668176799 NOTE, the test is temporarily disabled in https://github.com/apache/incubator-tvm/pull/6183, we will update this issue again once @vinx13 fix the problem

[GitHub] [incubator-tvm] merrymercy commented on a change in pull request #6194: [TIR] Enhance VerifyGPUCode

2020-08-03 Thread GitBox
merrymercy commented on a change in pull request #6194: URL: https://github.com/apache/incubator-tvm/pull/6194#discussion_r464592534 ## File path: src/tir/analysis/verify_gpu_code.cc ## @@ -134,25 +138,28 @@ class GPUCodeVerifier : public StmtExprVisitor { } } +

[incubator-tvm] branch master updated (21e895c -> a0c072e)

2020-08-03 Thread marisa
This is an automated email from the ASF dual-hosted git repository. marisa pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-tvm.git. from 21e895c [TIR] Enhance VerifyGPUCode (#6194) add a0c072e [TIR][Transform] HoistIfThenElse added (#6066)

[GitHub] [incubator-tvm] MarisaKirisame merged pull request #6066: [TIR][Transform] HoistIfThenElse added

2020-08-03 Thread GitBox
MarisaKirisame merged pull request #6066: URL: https://github.com/apache/incubator-tvm/pull/6066 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[incubator-tvm] branch master updated: [Fix] avoid unexpected throw in AttrInitEntry (#6128)

2020-08-03 Thread tqchen
This is an automated email from the ASF dual-hosted git repository. tqchen 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 84e3c19 [Fix] avoid unexpected throw in

[GitHub] [incubator-tvm] tqchen closed issue #6093: [BUG] Exception may be throwed when attrs is constructed

2020-08-03 Thread GitBox
tqchen closed issue #6093: URL: https://github.com/apache/incubator-tvm/issues/6093 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] tqchen merged pull request #6128: [Fix] avoid unexpected throw in AttrInitEntry

2020-08-03 Thread GitBox
tqchen merged pull request #6128: URL: https://github.com/apache/incubator-tvm/pull/6128 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] tqchen commented on pull request #6128: [Fix] avoid unexpected throw in AttrInitEntry

2020-08-03 Thread GitBox
tqchen commented on pull request #6128: URL: https://github.com/apache/incubator-tvm/pull/6128#issuecomment-668201707 Thanks @hanzz2007 ! This is an automated message from the Apache Git Service. To respond to the message,

[incubator-tvm] branch master updated (84e3c19 -> 21e895c)

2020-08-03 Thread lmzheng
This is an automated email from the ASF dual-hosted git repository. lmzheng pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-tvm.git. from 84e3c19 [Fix] avoid unexpected throw in AttrInitEntry (#6128) add 21e895c [TIR] Enhance VerifyGPUCode

[GitHub] [incubator-tvm] mbrookhart commented on pull request #6152: [RELAY] Basic block normal form

2020-08-03 Thread GitBox
mbrookhart commented on pull request #6152: URL: https://github.com/apache/incubator-tvm/pull/6152#issuecomment-668237667 > I dont think it break single responsibility - the code is doing conversion and a single configuration param denote which conversion it is. Another way to think about

[GitHub] [incubator-tvm] zhiics commented on pull request #6193: [DOCS] Update pass infra tutorial

2020-08-03 Thread GitBox
zhiics commented on pull request #6193: URL: https://github.com/apache/incubator-tvm/pull/6193#issuecomment-668192811 @tqchen I moved the debugging section to the the tutorial and updated the tutorial correspondingly. Since there are examples using different ways (including sequential) to

[GitHub] [incubator-tvm] mbrookhart edited a comment on pull request #6152: [RELAY] Basic block normal form

2020-08-03 Thread GitBox
mbrookhart edited a comment on pull request #6152: URL: https://github.com/apache/incubator-tvm/pull/6152#issuecomment-668237667 > I dont think it break single responsibility - the code is doing conversion and a single configuration param denote which conversion it is. Another way to

[GitHub] [incubator-tvm] MarisaKirisame commented on pull request #6152: [RELAY] Basic block normal form

2020-08-03 Thread GitBox
MarisaKirisame commented on pull request #6152: URL: https://github.com/apache/incubator-tvm/pull/6152#issuecomment-668238459 We can allow it if needed! This is an automated message from the Apache Git Service. To respond to

[GitHub] [incubator-tvm] mbrookhart commented on a change in pull request #6152: [RELAY] Basic block normal form

2020-08-03 Thread GitBox
mbrookhart commented on a change in pull request #6152: URL: https://github.com/apache/incubator-tvm/pull/6152#discussion_r464610109 ## File path: src/relay/transforms/to_basic_block_normal_form.cc ## @@ -0,0 +1,104 @@ +/* + * Licensed to the Apache Software Foundation (ASF)

[GitHub] [incubator-tvm] zhiics commented on a change in pull request #6162: [Parser] Parser 2.0 part 2

2020-08-03 Thread GitBox
zhiics commented on a change in pull request #6162: URL: https://github.com/apache/incubator-tvm/pull/6162#discussion_r464201530 ## File path: src/parser/meta_ref.cc ## @@ -0,0 +1,104 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more contributor

[GitHub] [incubator-tvm] eric-haibin-lin commented on a change in pull request #6152: [RELAY] Basic block normal form

2020-08-03 Thread GitBox
eric-haibin-lin commented on a change in pull request #6152: URL: https://github.com/apache/incubator-tvm/pull/6152#discussion_r464588608 ## File path: src/relay/transforms/to_a_normal_form.cc ## @@ -81,171 +70,187 @@ std::unordered_map CalcScope(const DependencyGrap

[GitHub] [incubator-tvm] electriclilies commented on a change in pull request #6198: [Relay][Dynamic] Add Dynamic Resize Op

2020-08-03 Thread GitBox
electriclilies commented on a change in pull request #6198: URL: https://github.com/apache/incubator-tvm/pull/6198#discussion_r464588897 ## File path: src/relay/op/dyn/image/resize.cc ## @@ -0,0 +1,106 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + *

[GitHub] [incubator-tvm] kevinthesun commented on pull request #6066: [TIR][Transform] HoistIfThenElse added

2020-08-03 Thread GitBox
kevinthesun commented on pull request #6066: URL: https://github.com/apache/incubator-tvm/pull/6066#issuecomment-668224314 @ANSHUMAN87 Thanks for clarification. Though we might not need to do so in this PR, it would be great if we can bring this in since AFAIK gpu is the major case for

[GitHub] [incubator-tvm] MarisaKirisame commented on pull request #6152: [RELAY] Basic block normal form

2020-08-03 Thread GitBox
MarisaKirisame commented on pull request #6152: URL: https://github.com/apache/incubator-tvm/pull/6152#issuecomment-668234290 I dont think it break single responsibility - the code is doing conversion and a single configuration param denote which conversion it is. Another way to think

[GitHub] [incubator-tvm] MarisaKirisame commented on pull request #6066: [TIR][Transform] HoistIfThenElse added

2020-08-03 Thread GitBox
MarisaKirisame commented on pull request #6066: URL: https://github.com/apache/incubator-tvm/pull/6066#issuecomment-668234871 Thanks @ANSHUMAN87 @roastduck @kevinthesun @kparzysz-quic @hzfan This is an automated message

[GitHub] [incubator-tvm] masahi commented on pull request #6200: [AutoQ] Additional pre-requisite passes for TF graphs

2020-08-03 Thread GitBox
masahi commented on pull request #6200: URL: https://github.com/apache/incubator-tvm/pull/6200#issuecomment-668296226 Better to add a test This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [incubator-tvm] jcf94 commented on a change in pull request #6190: [Ansor][AutoTVM v2.0] Phase 1: feature extraction for cost models

2020-08-03 Thread GitBox
jcf94 commented on a change in pull request #6190: URL: https://github.com/apache/incubator-tvm/pull/6190#discussion_r464756899 ## File path: src/auto_scheduler/feature.cc ## @@ -0,0 +1,1558 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more

[GitHub] [incubator-tvm] tqchen commented on pull request #6167: [TFLite] Implemented PADV2 Operator for TFLite and added support for constant values in PAD.

2020-08-03 Thread GitBox
tqchen commented on pull request #6167: URL: https://github.com/apache/incubator-tvm/pull/6167#issuecomment-668336940 THanks @jainris @leandron ! This is an automated message from the Apache Git Service. To respond to the

[GitHub] [incubator-tvm] tqchen merged pull request #6167: [TFLite] Implemented PADV2 Operator for TFLite and added support for constant values in PAD.

2020-08-03 Thread GitBox
tqchen merged pull request #6167: URL: https://github.com/apache/incubator-tvm/pull/6167 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

[incubator-tvm] branch master updated (d9c4f82 -> 90bde33)

2020-08-03 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 d9c4f82 [CI][ETHOSN] Enable CI for Ethos-N (#6171) add 90bde33 [TFLite] Implemented PADV2 Operator for

[GitHub] [incubator-tvm] 12101111 commented on issue #6197: [Rust] TVMError: Check failed: type_code_ == kDLInt (1 vs. 0) : expected int but get uint

2020-08-03 Thread GitBox
1210 commented on issue #6197: URL: https://github.com/apache/incubator-tvm/issues/6197#issuecomment-668367782 This example code is incorrect. ```rust // get the global TVM graph runtime function let runtime_create_fn =

[GitHub] [incubator-tvm] MarisaKirisame commented on pull request #6152: [RELAY] Basic block normal form

2020-08-03 Thread GitBox
MarisaKirisame commented on pull request #6152: URL: https://github.com/apache/incubator-tvm/pull/6152#issuecomment-668374973 Thanks @eric-haibin-lin @mbrookhart @junrushao1994 This is an automated message from the Apache

[GitHub] [incubator-tvm] mbrookhart commented on a change in pull request #6198: [Relay][Dynamic] Add Dynamic Resize Op

2020-08-03 Thread GitBox
mbrookhart commented on a change in pull request #6198: URL: https://github.com/apache/incubator-tvm/pull/6198#discussion_r464643632 ## File path: src/relay/op/dyn/image/resize.cc ## @@ -0,0 +1,106 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or

[GitHub] [incubator-tvm] comaniac commented on a change in pull request #6187: [Ansor][AutoTVM v2.0] Phase 1: The base class for cost models

2020-08-03 Thread GitBox
comaniac commented on a change in pull request #6187: URL: https://github.com/apache/incubator-tvm/pull/6187#discussion_r464690412 ## File path: include/tvm/auto_scheduler/cost_model.h ## @@ -0,0 +1,160 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + *

[GitHub] [incubator-tvm] anijain2305 opened a new pull request #6200: [AutoQ] Additional pre-requisite passes for TF graphs

2020-08-03 Thread GitBox
anijain2305 opened a new pull request #6200: URL: https://github.com/apache/incubator-tvm/pull/6200 @vinx13 @masahi I am trying TF graphs and found compilation failures while quantizing. Please review! This is

[GitHub] [incubator-tvm] tqchen merged pull request #6174: [DOCS] Added casting to hybrid script doc and fixed pass infra doc

2020-08-03 Thread GitBox
tqchen merged pull request #6174: URL: https://github.com/apache/incubator-tvm/pull/6174 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

[incubator-tvm] branch master updated (a0c072e -> 0e1f816)

2020-08-03 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 a0c072e [TIR][Transform] HoistIfThenElse added (#6066) add 0e1f816 [Target] Rename target_id =>

[GitHub] [incubator-tvm] tqchen commented on pull request #6174: [DOCS] Added casting to hybrid script doc and fixed pass infra doc

2020-08-03 Thread GitBox
tqchen commented on pull request #6174: URL: https://github.com/apache/incubator-tvm/pull/6174#issuecomment-668331487 Thanks @electriclilies @zhiics ! This is an automated message from the Apache Git Service. To respond to

[incubator-tvm] branch master updated (394bc33 -> 02b36e7)

2020-08-03 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 394bc33 [AutoScheduler] Fix alignment of note (#6181) add 02b36e7 [DOCS] Added casting to hybrid

[GitHub] [incubator-tvm] merrymercy edited a comment on pull request #6187: [Ansor][AutoTVM v2.0] Phase 1: The base class for cost models

2020-08-03 Thread GitBox
merrymercy edited a comment on pull request #6187: URL: https://github.com/apache/incubator-tvm/pull/6187#issuecomment-667658966 cc @junrushao1994 @comaniac @tqchen @jcf94 @ZihengJiang This is an automated message from the

[GitHub] [incubator-tvm] zeyu-hello opened a new pull request #6202: doc: change llvm download link to github for fast downloading

2020-08-03 Thread GitBox
zeyu-hello opened a new pull request #6202: URL: https://github.com/apache/incubator-tvm/pull/6202 Thanks for contributing to TVM! Please refer to guideline https://tvm.apache.org/docs/contribute/ for useful information and tips. After the pull request is submitted, please request code

[GitHub] [incubator-tvm] anijain2305 commented on pull request #6200: [AutoQ] Additional pre-requisite passes for TF graphs

2020-08-03 Thread GitBox
anijain2305 commented on pull request #6200: URL: https://github.com/apache/incubator-tvm/pull/6200#issuecomment-668296937 I was thinking of that too. But not sure what type of tests I can add. There are nightly MXNet tests.

[GitHub] [incubator-tvm] anijain2305 edited a comment on pull request #6200: [AutoQ] Additional pre-requisite passes for TF graphs

2020-08-03 Thread GitBox
anijain2305 edited a comment on pull request #6200: URL: https://github.com/apache/incubator-tvm/pull/6200#issuecomment-668296937 I was thinking of that too. But not sure what type of tests I can add. There are nightly MXNet tests. Any suggestion?

[GitHub] [incubator-tvm] merrymercy commented on a change in pull request #6187: [Ansor][AutoTVM v2.0] Phase 1: The base class for cost models

2020-08-03 Thread GitBox
merrymercy commented on a change in pull request #6187: URL: https://github.com/apache/incubator-tvm/pull/6187#discussion_r464750242 ## File path: include/tvm/auto_scheduler/cost_model.h ## @@ -0,0 +1,160 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one +

[incubator-tvm] branch master updated (0e1f816 -> 394bc33)

2020-08-03 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 0e1f816 [Target] Rename target_id => target_kind (#6199) add 394bc33 [AutoScheduler] Fix alignment of

[GitHub] [incubator-tvm] tqchen merged pull request #6199: [Target] Rename target_id => target_kind

2020-08-03 Thread GitBox
tqchen merged pull request #6199: URL: https://github.com/apache/incubator-tvm/pull/6199 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] tqchen merged pull request #6181: [AutoScheduler] Fix alignment of note

2020-08-03 Thread GitBox
tqchen merged pull request #6181: URL: https://github.com/apache/incubator-tvm/pull/6181 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] jcf94 commented on a change in pull request #6190: [Ansor][AutoTVM v2.0] Phase 1: feature extraction for cost models

2020-08-03 Thread GitBox
jcf94 commented on a change in pull request #6190: URL: https://github.com/apache/incubator-tvm/pull/6190#discussion_r46479 ## File path: include/tvm/auto_scheduler/feature.h ## @@ -0,0 +1,122 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or

[incubator-tvm] branch master updated (02b36e7 -> d9c4f82)

2020-08-03 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 02b36e7 [DOCS] Added casting to hybrid script doc and fixed pass infra doc (#6174) add d9c4f82

[GitHub] [incubator-tvm] tqchen commented on pull request #6143: [Relay] support i64 indices

2020-08-03 Thread GitBox
tqchen commented on pull request #6143: URL: https://github.com/apache/incubator-tvm/pull/6143#issuecomment-668335342 @hzfan is this PR ready for review? cc @yzhliu please followup This is an automated message from the

[GitHub] [incubator-tvm] comaniac commented on a change in pull request #6190: [Ansor][AutoTVM v2.0] Phase 1: feature extraction for cost models

2020-08-03 Thread GitBox
comaniac commented on a change in pull request #6190: URL: https://github.com/apache/incubator-tvm/pull/6190#discussion_r464720927 ## File path: include/tvm/auto_scheduler/feature.h ## @@ -0,0 +1,122 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or

[GitHub] [incubator-tvm] tqchen commented on pull request #6171: [CI][ETHOSN] Enable CI for Ethos-N

2020-08-03 Thread GitBox
tqchen commented on pull request #6171: URL: https://github.com/apache/incubator-tvm/pull/6171#issuecomment-668335709 Thanks @mbaret @zhiics ! This PR is merged. I will send another note when the docker binary get updated.

[GitHub] [incubator-tvm] tqchen merged pull request #6171: [CI][ETHOSN] Enable CI for Ethos-N

2020-08-03 Thread GitBox
tqchen merged pull request #6171: URL: https://github.com/apache/incubator-tvm/pull/6171 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

[incubator-tvm] branch master updated: [RELAY] Basic block normal form (#6152)

2020-08-03 Thread marisa
This is an automated email from the ASF dual-hosted git repository. marisa 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 b6db7e3 [RELAY] Basic block normal form

[GitHub] [incubator-tvm] MarisaKirisame merged pull request #6152: [RELAY] Basic block normal form

2020-08-03 Thread GitBox
MarisaKirisame merged pull request #6152: URL: https://github.com/apache/incubator-tvm/pull/6152 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [incubator-tvm] masahi edited a comment on pull request #6200: [AutoQ] Additional pre-requisite passes for TF graphs

2020-08-03 Thread GitBox
masahi edited a comment on pull request #6200: URL: https://github.com/apache/incubator-tvm/pull/6200#issuecomment-668298292 How about creating a simple NHWC relay graph by hand and doing dummy quantization? Just to make sure it doesn't raise error.

[GitHub] [incubator-tvm] masahi commented on pull request #6200: [AutoQ] Additional pre-requisite passes for TF graphs

2020-08-03 Thread GitBox
masahi commented on pull request #6200: URL: https://github.com/apache/incubator-tvm/pull/6200#issuecomment-668298292 How about creating a simple NHWC relay graph by hand and doing dummy quantization? Just to make sure it doesn't rais error.

[GitHub] [incubator-tvm] MarisaKirisame commented on pull request #6152: [RELAY] Basic block normal form

2020-08-03 Thread GitBox
MarisaKirisame commented on pull request #6152: URL: https://github.com/apache/incubator-tvm/pull/6152#issuecomment-668303155 @eric-haibin-lin please at me when it is done. I will merge it then. This is an automated message

[GitHub] [incubator-tvm] jcf94 commented on a change in pull request #6190: [Ansor][AutoTVM v2.0] Phase 1: feature extraction for cost models

2020-08-03 Thread GitBox
jcf94 commented on a change in pull request #6190: URL: https://github.com/apache/incubator-tvm/pull/6190#discussion_r464757334 ## File path: src/auto_scheduler/feature.cc ## @@ -0,0 +1,1558 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more

[GitHub] [incubator-tvm] jcf94 commented on a change in pull request #6190: [Ansor][AutoTVM v2.0] Phase 1: feature extraction for cost models

2020-08-03 Thread GitBox
jcf94 commented on a change in pull request #6190: URL: https://github.com/apache/incubator-tvm/pull/6190#discussion_r464757105 ## File path: src/auto_scheduler/feature.cc ## @@ -0,0 +1,1558 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more

[GitHub] [incubator-tvm] jcf94 commented on a change in pull request #6190: [Ansor][AutoTVM v2.0] Phase 1: feature extraction for cost models

2020-08-03 Thread GitBox
jcf94 commented on a change in pull request #6190: URL: https://github.com/apache/incubator-tvm/pull/6190#discussion_r464757285 ## File path: src/auto_scheduler/feature.cc ## @@ -0,0 +1,1558 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more

[GitHub] [incubator-tvm] wjliu1998 opened a new issue #6201: GoogleNet aux is None in pytorch test

2020-08-03 Thread GitBox
wjliu1998 opened a new issue #6201: URL: https://github.com/apache/incubator-tvm/issues/6201 When running `TVM_FFI=ctypes python -m pytest -v /home/liuwenjian/incubator-tvm/tests/python/frontend/pytorch/test_forward.py -k "test_googlenet" ` error will occur at

[GitHub] [incubator-tvm] kevinthesun commented on pull request #6066: [TIR][Transform] HoistIfThenElse added

2020-08-03 Thread GitBox
kevinthesun commented on pull request #6066: URL: https://github.com/apache/incubator-tvm/pull/6066#issuecomment-667826737 > > One thing I think it is good to have in this PR is to get some benchmark data, since we now enable this pass by default. IMO this pass is especially valuable when

[GitHub] [incubator-tvm] anijain2305 opened a new pull request #6195: [FoldConstant] Create Interpreter for each constant subgraph

2020-08-03 Thread GitBox
anijain2305 opened a new pull request #6195: URL: https://github.com/apache/incubator-tvm/pull/6195 This is related to https://discuss.tvm.ai/t/vm-slow-compilation-of-tf-object-detection-models/7479 For TF object detection models, a module has many functions (TF SSD mobilenet has

[GitHub] [incubator-tvm] jcf94 commented on a change in pull request #6194: [TIR] Enhance VerifyGPUCode

2020-08-03 Thread GitBox
jcf94 commented on a change in pull request #6194: URL: https://github.com/apache/incubator-tvm/pull/6194#discussion_r464250287 ## File path: src/tir/analysis/verify_gpu_code.cc ## @@ -134,25 +138,28 @@ class GPUCodeVerifier : public StmtExprVisitor { } } + void

[GitHub] [incubator-tvm] merrymercy opened a new pull request #6194: [TIR] Enhance VerifyGPUCode

2020-08-03 Thread GitBox
merrymercy opened a new pull request #6194: URL: https://github.com/apache/incubator-tvm/pull/6194 - Improve vectorization check in VerifyGPUCode - Add vthread length check in VerifyGPUCode cc @jcf94 @FrozenGene @csullivan @larrywyang