[GitHub] [incubator-tvm] masahi commented on a change in pull request #4497: [WIP] [Relay] Add a PyTorch to Relay Parser

2020-01-27 Thread GitBox
masahi commented on a change in pull request #4497: [WIP] [Relay] Add a PyTorch to Relay Parser URL: https://github.com/apache/incubator-tvm/pull/4497#discussion_r369428504 ## File path: python/tvm/relay/frontend/pytorch.py ## @@ -0,0 +1,1138 @@ +# Licensed to the Apache

[GitHub] [incubator-tvm] masahi commented on a change in pull request #4497: [WIP] [Relay] Add a PyTorch to Relay Parser

2020-01-27 Thread GitBox
masahi commented on a change in pull request #4497: [WIP] [Relay] Add a PyTorch to Relay Parser URL: https://github.com/apache/incubator-tvm/pull/4497#discussion_r369430546 ## File path: python/tvm/relay/frontend/pytorch.py ## @@ -0,0 +1,1138 @@ +# Licensed to the Apache

[GitHub] [incubator-tvm] masahi commented on issue #4497: [WIP] [Relay] Add a PyTorch to Relay Parser

2020-01-27 Thread GitBox
masahi commented on issue #4497: [WIP] [Relay] Add a PyTorch to Relay Parser URL: https://github.com/apache/incubator-tvm/pull/4497#issuecomment-579038361 @alexwong I'm 100% for "make all of the simpler changes to get this merged first". I don't want you to spend your time redoing the

[GitHub] [incubator-tvm] alexwong commented on issue #4497: [WIP] [Relay] Add a PyTorch to Relay Parser

2020-01-27 Thread GitBox
alexwong commented on issue #4497: [WIP] [Relay] Add a PyTorch to Relay Parser URL: https://github.com/apache/incubator-tvm/pull/4497#issuecomment-579028784 > @alexwong I had a big success in refactoring, the parser itself (except op conversion) is about 150 lines and the "main loop" is

[GitHub] [incubator-tvm] jroesch commented on issue #4782: [PassManager] Implement pass manager tracing API

2020-01-27 Thread GitBox
jroesch commented on issue #4782: [PassManager] Implement pass manager tracing API URL: https://github.com/apache/incubator-tvm/pull/4782#issuecomment-579028693 @tmoreau89 I added to the pass docs with an example. This is

[GitHub] [incubator-tvm] jroesch commented on issue #4782: [PassManager] Implement pass manager tracing API

2020-01-27 Thread GitBox
jroesch commented on issue #4782: [PassManager] Implement pass manager tracing API URL: https://github.com/apache/incubator-tvm/pull/4782#issuecomment-579026027 cc @zhiics @tmoreau89 This is an automated message from the

[GitHub] [incubator-tvm] jroesch opened a new pull request #4782: [PassManager] Implement pass manager tracing API

2020-01-27 Thread GitBox
jroesch opened a new pull request #4782: [PassManager] Implement pass manager tracing API URL: https://github.com/apache/incubator-tvm/pull/4782 I am working on some pass infrastructure for TVM and needed the previously discussed tracing

[incubator-tvm] branch pass_callback_via_cx created (now 00ec7f9)

2020-01-27 Thread jroesch
This is an automated email from the ASF dual-hosted git repository. jroesch pushed a change to branch pass_callback_via_cx in repository https://gitbox.apache.org/repos/asf/incubator-tvm.git. at 00ec7f9 [Build] Explicitly link to cublasLt if it exists (#4776) No new revisions were added

[incubator-tvm] branch pass_callback_via_cx updated: Implement pass tracing API

2020-01-27 Thread jroesch
This is an automated email from the ASF dual-hosted git repository. jroesch pushed a commit to branch pass_callback_via_cx in repository https://gitbox.apache.org/repos/asf/incubator-tvm.git The following commit(s) were added to refs/heads/pass_callback_via_cx by this push: new 1f38493

[incubator-tvm] branch master updated (de919cb -> d54036a)

2020-01-27 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 de919cb [Relay][Frontend][ONNX] Broadcast condition, x, and y for Where op (#4774) add d54036a Safe

[GitHub] [incubator-tvm] merrymercy merged pull request #4781: [AutoTVM] Ignore error when removing tmpdir

2020-01-27 Thread GitBox
merrymercy merged pull request #4781: [AutoTVM] Ignore error when removing tmpdir URL: https://github.com/apache/incubator-tvm/pull/4781 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [incubator-tvm] comaniac commented on issue #4771: [Relay] Added Merge Composite pass

2020-01-27 Thread GitBox
comaniac commented on issue #4771: [Relay] Added Merge Composite pass URL: https://github.com/apache/incubator-tvm/pull/4771#issuecomment-579014540 So you want the flow to be: ``` CompositeMerge -> Annotation -> Partitioning. ``` I agree that this would make annotation generic

[GitHub] [incubator-tvm] jroesch merged pull request #4774: [Relay][Frontend][ONNX] Broadcast condition, x, and y for Where op

2020-01-27 Thread GitBox
jroesch merged pull request #4774: [Relay][Frontend][ONNX] Broadcast condition, x, and y for Where op URL: https://github.com/apache/incubator-tvm/pull/4774 This is an automated message from the Apache Git Service. To

[incubator-tvm] branch master updated (f71a10c -> de919cb)

2020-01-27 Thread jroesch
This is an automated email from the ASF dual-hosted git repository. jroesch pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-tvm.git. from f71a10c properly extract error type from windows error message (#4780) add de919cb

[GitHub] [incubator-tvm] tqchen commented on issue #4780: [FFI][Windows] Fix hasattr by extracting Python error type from Windows error message

2020-01-27 Thread GitBox
tqchen commented on issue #4780: [FFI][Windows] Fix hasattr by extracting Python error type from Windows error message URL: https://github.com/apache/incubator-tvm/pull/4780#issuecomment-578996008 Thanks @jonso4 ! This is an

[incubator-tvm] branch master updated (00ec7f9 -> f71a10c)

2020-01-27 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 00ec7f9 [Build] Explicitly link to cublasLt if it exists (#4776) add f71a10c properly extract error

[GitHub] [incubator-tvm] tqchen merged pull request #4780: [FFI][Windows] Fix hasattr by extracting Python error type from Windows error message

2020-01-27 Thread GitBox
tqchen merged pull request #4780: [FFI][Windows] Fix hasattr by extracting Python error type from Windows error message URL: https://github.com/apache/incubator-tvm/pull/4780 This is an automated message from the Apache Git

[incubator-tvm] branch master updated (9056fc4 -> 00ec7f9)

2020-01-27 Thread masahi
This is an automated email from the ASF dual-hosted git repository. masahi pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-tvm.git. from 9056fc4 Update tune_simple_template.py (#4778) add 00ec7f9 [Build] Explicitly link to cublasLt if it

[GitHub] [incubator-tvm] masahi commented on issue #4776: [Build] Explicitly link to cublasLt if it exists

2020-01-27 Thread GitBox
masahi commented on issue #4776: [Build] Explicitly link to cublasLt if it exists URL: https://github.com/apache/incubator-tvm/pull/4776#issuecomment-578985825 thanks @soiferj This is an automated message from the Apache

[GitHub] [incubator-tvm] masahi merged pull request #4776: [Build] Explicitly link to cublasLt if it exists

2020-01-27 Thread GitBox
masahi merged pull request #4776: [Build] Explicitly link to cublasLt if it exists URL: https://github.com/apache/incubator-tvm/pull/4776 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [incubator-tvm] mbarrett97 commented on issue #4771: [Relay] Added Merge Composite pass

2020-01-27 Thread GitBox
mbarrett97 commented on issue #4771: [Relay] Added Merge Composite pass URL: https://github.com/apache/incubator-tvm/pull/4771#issuecomment-578979321 It's the intention that this can be called on the entire Relay graph so that it can be used to help implement a generic annotation pass (one

[GitHub] [incubator-tvm] mbarrett97 closed issue #4150: [RFC] [AutoTVM] Implementing an auto-tuning library/cache

2020-01-27 Thread GitBox
mbarrett97 closed issue #4150: [RFC] [AutoTVM] Implementing an auto-tuning library/cache URL: https://github.com/apache/incubator-tvm/issues/4150 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [incubator-tvm] mbarrett97 commented on a change in pull request #4771: [Relay] Added Merge Composite pass

2020-01-27 Thread GitBox
mbarrett97 commented on a change in pull request #4771: [Relay] Added Merge Composite pass URL: https://github.com/apache/incubator-tvm/pull/4771#discussion_r371508067 ## File path: src/relay/pass/merge_composite.cc ## @@ -0,0 +1,192 @@ +/* + * Licensed to the Apache

[GitHub] [incubator-tvm] comaniac commented on a change in pull request #4771: [Relay] Added Merge Composite pass

2020-01-27 Thread GitBox
comaniac commented on a change in pull request #4771: [Relay] Added Merge Composite pass URL: https://github.com/apache/incubator-tvm/pull/4771#discussion_r371504533 ## File path: src/relay/pass/merge_composite.cc ## @@ -0,0 +1,192 @@ +/* + * Licensed to the Apache

[GitHub] [incubator-tvm] comaniac opened a new pull request #4781: [AutoTVM] Ignore error when removing tmpdir

2020-01-27 Thread GitBox
comaniac opened a new pull request #4781: [AutoTVM] Ignore error when removing tmpdir URL: https://github.com/apache/incubator-tvm/pull/4781 Sometimes the tmpdir drops for some reasons. For example, /tmp is cleaned by someone or similar situations. Anyhow, it seems to me that the use of

[GitHub] [incubator-tvm] wpan11nv commented on a change in pull request #4779: [AUTOTVM] Fix a bug in generating the search space

2020-01-27 Thread GitBox
wpan11nv commented on a change in pull request #4779: [AUTOTVM] Fix a bug in generating the search space URL: https://github.com/apache/incubator-tvm/pull/4779#discussion_r371466338 ## File path: python/tvm/autotvm/task/space.py ## @@ -226,7 +226,13 @@ def __init__(self,

[GitHub] [incubator-tvm] abergeron commented on issue #4758: [relay] ADT match becomes not well formed during VM optimization

2020-01-27 Thread GitBox
abergeron commented on issue #4758: [relay] ADT match becomes not well formed during VM optimization URL: https://github.com/apache/incubator-tvm/issues/4758#issuecomment-578918024 After some experimentation, if I comment out InlinePrimitives at line 921 in compiler.cc

[GitHub] [incubator-tvm] soiferj opened a new pull request #4780: [FFI][Windows] Extract Python error type from Windows error log

2020-01-27 Thread GitBox
soiferj opened a new pull request #4780: [FFI][Windows] Extract Python error type from Windows error log URL: https://github.com/apache/incubator-tvm/pull/4780 As discussed

[GitHub] [incubator-tvm] soiferj commented on issue #4776: [Build] Explicitly link to cublasLt if it exists

2020-01-27 Thread GitBox
soiferj commented on issue #4776: [Build] Explicitly link to cublasLt if it exists URL: https://github.com/apache/incubator-tvm/pull/4776#issuecomment-578902685 @masahi fixed. Would you mind taking another look? This is an

[GitHub] [incubator-tvm] comaniac commented on a change in pull request #4779: [AUTOTVM] Fix a bug in generating the search space

2020-01-27 Thread GitBox
comaniac commented on a change in pull request #4779: [AUTOTVM] Fix a bug in generating the search space URL: https://github.com/apache/incubator-tvm/pull/4779#discussion_r371422539 ## File path: python/tvm/autotvm/task/space.py ## @@ -226,7 +226,13 @@ def __init__(self,

[GitHub] [incubator-tvm] comaniac commented on a change in pull request #4779: [AUTOTVM] Fix a bug in generating the search space

2020-01-27 Thread GitBox
comaniac commented on a change in pull request #4779: [AUTOTVM] Fix a bug in generating the search space URL: https://github.com/apache/incubator-tvm/pull/4779#discussion_r371422539 ## File path: python/tvm/autotvm/task/space.py ## @@ -226,7 +226,13 @@ def __init__(self,

[GitHub] [incubator-tvm] wpan11nv commented on a change in pull request #4779: [AUTOTVM] Fix a bug in generating the search space

2020-01-27 Thread GitBox
wpan11nv commented on a change in pull request #4779: [AUTOTVM] Fix a bug in generating the search space URL: https://github.com/apache/incubator-tvm/pull/4779#discussion_r371417185 ## File path: python/tvm/autotvm/task/space.py ## @@ -226,7 +226,13 @@ def __init__(self,

[GitHub] [incubator-tvm] comaniac commented on a change in pull request #4779: [AUTOTVM] Fix a bug in generating the search space

2020-01-27 Thread GitBox
comaniac commented on a change in pull request #4779: [AUTOTVM] Fix a bug in generating the search space URL: https://github.com/apache/incubator-tvm/pull/4779#discussion_r371397521 ## File path: python/tvm/autotvm/task/space.py ## @@ -226,7 +226,13 @@ def __init__(self,

[GitHub] [incubator-tvm] wpan11nv opened a new pull request #4779: [AUTOTVM] Fix a bug in generating the search space

2020-01-27 Thread GitBox
wpan11nv opened a new pull request #4779: [AUTOTVM] Fix a bug in generating the search space URL: https://github.com/apache/incubator-tvm/pull/4779 - Do not use numpy.prod which ignores integer (64 bits) overflows. This leads to an incorrect number of points in the search space.

[GitHub] [incubator-tvm] yongfeng-nv commented on issue #4651: Tensor Expression Debug Display (TEDD)

2020-01-27 Thread GitBox
yongfeng-nv commented on issue #4651: Tensor Expression Debug Display (TEDD) URL: https://github.com/apache/incubator-tvm/pull/4651#issuecomment-578845333 > > Thanks @yongfeng-nv . One thing that I think worth considering, as in many viz tools, is the separation of visualization data

[GitHub] [incubator-tvm] FrozenGene commented on a change in pull request #4543: [FRONTEND][TFLITE] Add support for TFLite_Detection_PostProcess

2020-01-27 Thread GitBox
FrozenGene commented on a change in pull request #4543: [FRONTEND][TFLITE] Add support for TFLite_Detection_PostProcess URL: https://github.com/apache/incubator-tvm/pull/4543#discussion_r371281677 ## File path: tests/python/frontend/tflite/test_forward.py ## @@ -1113,6

[GitHub] [incubator-tvm] FrozenGene commented on a change in pull request #4543: [FRONTEND][TFLITE] Add support for TFLite_Detection_PostProcess

2020-01-27 Thread GitBox
FrozenGene commented on a change in pull request #4543: [FRONTEND][TFLITE] Add support for TFLite_Detection_PostProcess URL: https://github.com/apache/incubator-tvm/pull/4543#discussion_r371281677 ## File path: tests/python/frontend/tflite/test_forward.py ## @@ -1113,6

[GitHub] [incubator-tvm] mbarrett97 commented on a change in pull request #4543: [FRONTEND][TFLITE] Add support for TFLite_Detection_PostProcess

2020-01-27 Thread GitBox
mbarrett97 commented on a change in pull request #4543: [FRONTEND][TFLITE] Add support for TFLite_Detection_PostProcess URL: https://github.com/apache/incubator-tvm/pull/4543#discussion_r371276909 ## File path: tests/python/frontend/tflite/test_forward.py ## @@ -1113,6

[GitHub] [incubator-tvm] FrozenGene commented on a change in pull request #4695: [Relay][Frontend][TFlite] Add parser support for relational ops

2020-01-27 Thread GitBox
FrozenGene commented on a change in pull request #4695: [Relay][Frontend][TFlite] Add parser support for relational ops URL: https://github.com/apache/incubator-tvm/pull/4695#discussion_r371276409 ## File path: python/tvm/relay/frontend/tflite.py ## @@ -705,47 +710,77 @@

[GitHub] [incubator-tvm] FrozenGene commented on a change in pull request #4543: [FRONTEND][TFLITE] Add support for TFLite_Detection_PostProcess

2020-01-27 Thread GitBox
FrozenGene commented on a change in pull request #4543: [FRONTEND][TFLITE] Add support for TFLite_Detection_PostProcess URL: https://github.com/apache/incubator-tvm/pull/4543#discussion_r371269023 ## File path: tests/python/frontend/tflite/test_forward.py ## @@ -1113,6

[GitHub] [incubator-tvm] mbarrett97 commented on a change in pull request #4543: [FRONTEND][TFLITE] Add support for TFLite_Detection_PostProcess

2020-01-27 Thread GitBox
mbarrett97 commented on a change in pull request #4543: [FRONTEND][TFLITE] Add support for TFLite_Detection_PostProcess URL: https://github.com/apache/incubator-tvm/pull/4543#discussion_r371205476 ## File path: tests/python/frontend/tflite/test_forward.py ## @@ -1113,6

[GitHub] [incubator-tvm] inadob commented on a change in pull request #4695: [Relay][Frontend][TFlite] Add parser support for relational ops

2020-01-27 Thread GitBox
inadob commented on a change in pull request #4695: [Relay][Frontend][TFlite] Add parser support for relational ops URL: https://github.com/apache/incubator-tvm/pull/4695#discussion_r371203978 ## File path: python/tvm/relay/frontend/tflite.py ## @@ -705,47 +710,77 @@ def

[GitHub] [incubator-tvm] FrozenGene commented on a change in pull request #4543: [FRONTEND][TFLITE] Add support for TFLite_Detection_PostProcess

2020-01-27 Thread GitBox
FrozenGene commented on a change in pull request #4543: [FRONTEND][TFLITE] Add support for TFLite_Detection_PostProcess URL: https://github.com/apache/incubator-tvm/pull/4543#discussion_r371193247 ## File path: tests/python/frontend/tflite/test_forward.py ## @@ -1113,6

[GitHub] [incubator-tvm] mbarrett97 commented on a change in pull request #4543: [FRONTEND][TFLITE] Add support for TFLite_Detection_PostProcess

2020-01-27 Thread GitBox
mbarrett97 commented on a change in pull request #4543: [FRONTEND][TFLITE] Add support for TFLite_Detection_PostProcess URL: https://github.com/apache/incubator-tvm/pull/4543#discussion_r371170921 ## File path: tests/python/frontend/tflite/test_forward.py ## @@ -1113,6

[GitHub] [incubator-tvm] hcho3 commented on issue #4412: Python binding: No module named 'topi'

2020-01-27 Thread GitBox
hcho3 commented on issue #4412: Python binding: No module named 'topi' URL: https://github.com/apache/incubator-tvm/issues/4412#issuecomment-578689513 I've had a similar problem before, and the solution is to run `python setup.py install` from `topi/python` directory. This command installs

[GitHub] [incubator-tvm] leandron commented on issue #4756: [Docker] Update torch version to 1.4

2020-01-27 Thread GitBox
leandron commented on issue #4756: [Docker] Update torch version to 1.4 URL: https://github.com/apache/incubator-tvm/pull/4756#issuecomment-578681838 Sorry @masahi , I don't know how that error is related to the Pillow dependency and couldn't find any related issue on sphinx.