liangfu commented on a change in pull request #4703: [VTA] Support network
which have no unique operator as start/stop name for graph pack.
URL: https://github.com/apache/incubator-tvm/pull/4703#discussion_r366724818
##
File path: vta/python/vta/top/graphpack.py
##
@@ -246
liangfu commented on a change in pull request #4703: [VTA] Support network
which have no unique operator as start/stop name for graph pack.
URL: https://github.com/apache/incubator-tvm/pull/4703#discussion_r366724607
##
File path: vta/python/vta/top/graphpack.py
##
@@ -246
masahi commented on issue #4497: [WIP] [Relay] Add a PyTorch to Relay Parser
URL: https://github.com/apache/incubator-tvm/pull/4497#issuecomment-574517425
ok I found a dirty hack to remove prim::CallMethod. Add
```torch._C._jit_pass_inline(trace.graph)```
after tracing. This will move
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 81e03ee Revert "[Relay][TOPI]Fix meaning of conv2d_transpose
output_padding parameter (#4318)" (#4708)
tqchen merged pull request #4713: link the math library by default
URL: https://github.com/apache/incubator-tvm/pull/4713
This is an automated message from the Apache Git Service.
To respond to the message, please log on to G
FrozenGene commented on issue #4713: link the math library by default
URL: https://github.com/apache/incubator-tvm/pull/4713#issuecomment-574487666
@tqchen could you help to merge it next? Thanks.
This is an automated message
FrozenGene commented on issue #4713: link the math library by default
URL: https://github.com/apache/incubator-tvm/pull/4713#issuecomment-574487438
Thanks @zhigaowu !
This is an automated message from the Apache Git Service.
T
zhigaowu opened a new pull request #4713: link the math library by default
URL: https://github.com/apache/incubator-tvm/pull/4713
Https://discuss.tvm.ai/t/cannot-locate-symbol-expf-error-on-android-host-system-ubuntu/5387
@FrozenGene link the math library by default
--
tqchen merged pull request #4708: Revert "[Relay][TOPI]Fix meaning of
conv2d_transpose output_padding parameter"
URL: https://github.com/apache/incubator-tvm/pull/4708
This is an automated message from the Apache Git Service
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 7f7dc07 use packed func macro for external codegen (#4710)
add 81e03ee Revert "[Relay][TOPI]Fix meaning
zhigaowu closed pull request #4712: V0.6
URL: https://github.com/apache/incubator-tvm/pull/4712
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
zhigaowu opened a new pull request #4712: V0.6
URL: https://github.com/apache/incubator-tvm/pull/4712
Https://discuss.tvm.ai/t/cannot-locate-symbol-expf-error-on-android-host-system-ubuntu/5387
@FrozenGene link the math library by default
--
yzhliu commented on issue #4710: [relay][external_codgen] use packed func macro
for external codegen
URL: https://github.com/apache/incubator-tvm/pull/4710#issuecomment-574480750
Thanks @zhiics @tqchen
This is an automated m
This is an automated email from the ASF dual-hosted git repository.
liuyizhi pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-tvm.git.
from ce807fe [REFACTOR][IR] Unify IntImm and UIntImm (#4706)
add 7f7dc07 use packed func macro for externa
yzhliu merged pull request #4710: [relay][external_codgen] use packed func
macro for external codegen
URL: https://github.com/apache/incubator-tvm/pull/4710
This is an automated message from the Apache Git Service.
To respon
liaha opened a new pull request #4711: [Relay][Frontend][TF] fix _parse_param
bug
URL: https://github.com/apache/incubator-tvm/pull/4711
* fix parsing bug that DT_STRING Const name might not be defined in shape
dictionary specified in from_tensorflow parameter
* update test case
--
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 f4c4fde [REFACTOR][IR] Polish ir/type (#4705)
add ce807fe [REFACTOR][IR] Unify IntImm and UIntImm (#470
tqchen merged pull request #4706: [REFACTOR][IR] Unify IntImm and UIntImm
URL: https://github.com/apache/incubator-tvm/pull/4706
This is an automated message from the Apache Git Service.
To respond to the message, please log
zhiics commented on a change in pull request #4602: [Docs] Bring Your Own
Codegen Guide -- Part 1
URL: https://github.com/apache/incubator-tvm/pull/4602#discussion_r35208
##
File path: docs/dev/relay_bring_your_own_codegen.rst
##
@@ -0,0 +1,529 @@
+.. Licensed to the
zhiics commented on issue #4710: [relay][external_codgen] use packed func macro
for external codegen
URL: https://github.com/apache/incubator-tvm/pull/4710#issuecomment-574461129
cc @tqchen @comaniac
This is an automated mes
zhiics opened a new pull request #4710: [relay][external_codgen] use packed
func macro for external codegen
URL: https://github.com/apache/incubator-tvm/pull/4710
#4602 Simplify the wrapper using `TVM_DLL_EXPORT_TYPED_FUNC` macro.
---
comaniac commented on a change in pull request #4602: [Docs] Bring Your Own
Codegen Guide -- Part 1
URL: https://github.com/apache/incubator-tvm/pull/4602#discussion_r366648208
##
File path: docs/dev/relay_bring_your_own_codegen.rst
##
@@ -0,0 +1,529 @@
+.. Licensed to th
tqchen opened a new pull request #4709: [REFACTOR][IR] attrs.h -> ir
URL: https://github.com/apache/incubator-tvm/pull/4709
This PR moves attrs.h into the ir folder as it
can serve as a common infra for building ir dats structures.
We also moved common container(FloatImm) into ir/e
tqchen commented on issue #4709: [REFACTOR][IR] attrs.h -> ir
URL: https://github.com/apache/incubator-tvm/pull/4709#issuecomment-574436966
cc @yzhliu @zhiics @ZihengJiang
This is an automated message from the Apache Git Serv
tqchen commented on a change in pull request #4602: [Docs] Bring Your Own
Codegen Guide -- Part 1
URL: https://github.com/apache/incubator-tvm/pull/4602#discussion_r366640373
##
File path: docs/dev/relay_bring_your_own_codegen.rst
##
@@ -0,0 +1,529 @@
+.. Licensed to the
tqchen commented on a change in pull request #4602: [Docs] Bring Your Own
Codegen Guide -- Part 1
URL: https://github.com/apache/incubator-tvm/pull/4602#discussion_r366639821
##
File path: docs/dev/relay_bring_your_own_codegen.rst
##
@@ -0,0 +1,529 @@
+.. Licensed to the
tqchen commented on a change in pull request #4602: [Docs] Bring Your Own
Codegen Guide -- Part 1
URL: https://github.com/apache/incubator-tvm/pull/4602#discussion_r366639821
##
File path: docs/dev/relay_bring_your_own_codegen.rst
##
@@ -0,0 +1,529 @@
+.. Licensed to the
icemelon9 edited a comment on issue #4318: [Relay][TOPI]Fix meaning of
conv2d_transpose output_padding parameter
URL: https://github.com/apache/incubator-tvm/pull/4318#issuecomment-574434976
@abergeron @vinx13 @tmoreau89
I found two problems in this PR.
1. In this
[line](https://gi
icemelon9 edited a comment on issue #4318: [Relay][TOPI]Fix meaning of
conv2d_transpose output_padding parameter
URL: https://github.com/apache/incubator-tvm/pull/4318#issuecomment-574434976
@abergeron @vinx13 @tmoreau89
I found out two problems in this PR.
1. In this
[line](https:
icemelon9 edited a comment on issue #4318: [Relay][TOPI]Fix meaning of
conv2d_transpose output_padding parameter
URL: https://github.com/apache/incubator-tvm/pull/4318#issuecomment-574434976
@abergeron @vinx13 @tmoreau89
I found out two problems in this PR.
1. In this
[line](https:
This is an automated email from the ASF dual-hosted git repository.
haichen pushed a commit to branch revert-4318-conv2d_transpose_fix
in repository https://gitbox.apache.org/repos/asf/incubator-tvm.git
commit ebfff4419fc714d173a1e365739badee1d79adfa
Author: Haichen Shen
AuthorDate: Tue Jan 14 1
icemelon9 opened a new pull request #4708: Revert "[Relay][TOPI]Fix meaning of
conv2d_transpose output_padding parameter"
URL: https://github.com/apache/incubator-tvm/pull/4708
Reverts apache/incubator-tvm#4318
Bugs in the PR.
This is an automated email from the ASF dual-hosted git repository.
haichen pushed a change to branch revert-4318-conv2d_transpose_fix
in repository https://gitbox.apache.org/repos/asf/incubator-tvm.git.
at ebfff44 Revert "[Relay][TOPI]Fix meaning of conv2d_transpose
output_padding param
icemelon9 commented on issue #4318: [Relay][TOPI]Fix meaning of
conv2d_transpose output_padding parameter
URL: https://github.com/apache/incubator-tvm/pull/4318#issuecomment-574434976
@abergeron @vinx13 @tmoreau89
I found out two problems in this PR.
1. In this
[line](https://githu
hcho3 opened a new pull request #4707: [Relay][Frontend][TF] Fix handling of 0D
scalar Constant
URL: https://github.com/apache/incubator-tvm/pull/4707
Consider the following snippet of a TensorFlow model:
```
node {
name: "flatten_1/stack/0"
op: "Const"
attr {
ke
comaniac commented on a change in pull request #4602: [Docs] Bring Your Own
Codegen Guide -- Part 1
URL: https://github.com/apache/incubator-tvm/pull/4602#discussion_r366634417
##
File path: docs/dev/relay_bring_your_own_codegen.rst
##
@@ -0,0 +1,529 @@
+.. Licensed to th
yongfeng-nv commented on issue #4651: Tensor Expression Debug Display (TEDD)
URL: https://github.com/apache/incubator-tvm/pull/4651#issuecomment-574416251
> Thanks @yongfeng-nv . One thing that I think worth considering, as in many
viz tools, is the separation of visualization data source s
tqchen commented on a change in pull request #4602: [Docs] Bring Your Own
Codegen Guide -- Part 1
URL: https://github.com/apache/incubator-tvm/pull/4602#discussion_r366616774
##
File path: docs/dev/relay_bring_your_own_codegen.rst
##
@@ -0,0 +1,529 @@
+.. Licensed to the
tqchen commented on a change in pull request #4602: [Docs] Bring Your Own
Codegen Guide -- Part 1
URL: https://github.com/apache/incubator-tvm/pull/4602#discussion_r366616774
##
File path: docs/dev/relay_bring_your_own_codegen.rst
##
@@ -0,0 +1,529 @@
+.. Licensed to the
tqchen commented on a change in pull request #4602: [Docs] Bring Your Own
Codegen Guide -- Part 1
URL: https://github.com/apache/incubator-tvm/pull/4602#discussion_r366615883
##
File path: docs/dev/relay_bring_your_own_codegen.rst
##
@@ -0,0 +1,529 @@
+.. Licensed to the
zhiics commented on a change in pull request #4706: [REFACTOR][IR] Unify IntImm
and UIntImm
URL: https://github.com/apache/incubator-tvm/pull/4706#discussion_r366610155
##
File path: include/tvm/expr_operator.h
##
@@ -597,6 +583,15 @@ TVM_DLL PrimExpr nearbyint(PrimExpr x)
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 3f2abfb [relay] Relay annotation and partitioning for external
compilers (#4570)
add f4c4fde [REFACTOR
tqchen merged pull request #4705: [REFACTOR][IR] Polish ir/type
URL: https://github.com/apache/incubator-tvm/pull/4705
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitH
masahi commented on issue #4497: [WIP] [Relay] Add a PyTorch to Relay Parser
URL: https://github.com/apache/incubator-tvm/pull/4497#issuecomment-574403934
I'm happy to merge this even if we can only support v1.3 models for now. I
want to send other op converters I need.
---
tqchen opened a new pull request #4706: [REFACTOR][IR] Unify IntImm and UIntImm
URL: https://github.com/apache/incubator-tvm/pull/4706
This PR unifies UIntImm and IntImm to simplify the codebase.
Unsigned integer constants will also be stored as IntImm.
For uint constant that does
yzhliu commented on issue #4684: [Arith] add ShapeVar representing non-neg
valued variable in a tensor shape
URL: https://github.com/apache/incubator-tvm/pull/4684#issuecomment-574397147
https://discuss.tvm.ai/t/rfc-name-for-non-negative-variable
---
comaniac commented on a change in pull request #4602: [Docs] Bring Your Own
Codegen Guide -- Part 1
URL: https://github.com/apache/incubator-tvm/pull/4602#discussion_r366589820
##
File path: docs/dev/relay_bring_your_own_codegen.rst
##
@@ -0,0 +1,529 @@
+.. Licensed to th
comaniac commented on a change in pull request #4602: [Docs] Bring Your Own
Codegen Guide -- Part 1
URL: https://github.com/apache/incubator-tvm/pull/4602#discussion_r366584308
##
File path: docs/dev/relay_bring_your_own_codegen.rst
##
@@ -0,0 +1,529 @@
+.. Licensed to th
alexwong commented on issue #4497: [WIP] [Relay] Add a PyTorch to Relay Parser
URL: https://github.com/apache/incubator-tvm/pull/4497#issuecomment-574383474
> @alexwong I tried your PR locally. With pytorch v1.3 it works, but they
introduced big change in
[#28408](https://github.com/pytorc
tqchen commented on a change in pull request #4602: [Docs] Bring Your Own
Codegen Guide -- Part 1
URL: https://github.com/apache/incubator-tvm/pull/4602#discussion_r366582394
##
File path: docs/dev/relay_bring_your_own_codegen.rst
##
@@ -0,0 +1,529 @@
+.. Licensed to the
tqchen commented on a change in pull request #4602: [Docs] Bring Your Own
Codegen Guide -- Part 1
URL: https://github.com/apache/incubator-tvm/pull/4602#discussion_r366581709
##
File path: docs/dev/relay_bring_your_own_codegen.rst
##
@@ -0,0 +1,529 @@
+.. Licensed to the
tqchen commented on a change in pull request #4602: [Docs] Bring Your Own
Codegen Guide -- Part 1
URL: https://github.com/apache/incubator-tvm/pull/4602#discussion_r366580897
##
File path: docs/dev/relay_bring_your_own_codegen.rst
##
@@ -0,0 +1,529 @@
+.. Licensed to the
tqchen commented on a change in pull request #4602: [Docs] Bring Your Own
Codegen Guide -- Part 1
URL: https://github.com/apache/incubator-tvm/pull/4602#discussion_r366580758
##
File path: docs/dev/relay_bring_your_own_codegen.rst
##
@@ -0,0 +1,529 @@
+.. Licensed to the
tqchen commented on issue #4684: [Arith] add ShapeVar representing non-neg
valued variable in a tensor shape
URL: https://github.com/apache/incubator-tvm/pull/4684#issuecomment-574380538
We can ask other's thoughts about it(perhaps list a few candidates and send
an rfc?). TIndex seems to b
masahi edited a comment on issue #4497: [WIP] [Relay] Add a PyTorch to Relay
Parser
URL: https://github.com/apache/incubator-tvm/pull/4497#issuecomment-574076780
@alexwong I tried your PR locally. With pytorch v1.3 it works, but they
introduced big change in
[#28408](https://github.com/py
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 d7d2a9b [REFACTOR][IR] Initialize Unified IR Pass Infra. (#4702)
add 3f2abfb [relay] Relay annotation and
zhiics commented on issue #4570: [relay] Relay annotation and partitioning for
external compilers
URL: https://github.com/apache/incubator-tvm/pull/4570#issuecomment-574338792
Thanks @tqchen @masahi @comaniac
This is an auto
zhiics merged pull request #4570: [relay] Relay annotation and partitioning for
external compilers
URL: https://github.com/apache/incubator-tvm/pull/4570
This is an automated message from the Apache Git Service.
To respond t
comaniac commented on issue #4602: [Docs] Bring Your Own Codegen Guide -- Part 1
URL: https://github.com/apache/incubator-tvm/pull/4602#issuecomment-574337997
Should we merge this part first if no one wants to further review it for now?
cc @masahi @tqchen
---
huajsj edited a comment on issue #4703: [VTA] Support network which have no
unique operator as start/stop name for graph pack.
URL: https://github.com/apache/incubator-tvm/pull/4703#issuecomment-574325187
Hi Liangfu, Thanks for the comments, the reason here to use operator name
and indices
zhiics commented on issue #4693: [runtime][refactor] Unify vm and interpreter
objects
URL: https://github.com/apache/incubator-tvm/pull/4693#issuecomment-574328741
ping reviewers.
This is an automated message from the Apache
huajsj commented on issue #4703: [VTA] Support network which have no unique
operator as start/stop name for graph pack.
URL: https://github.com/apache/incubator-tvm/pull/4703#issuecomment-574325187
Hi Liangfu, Thanks for the comments, the reason here to use operator name
and indices is to
yzhliu commented on issue #4684: [Arith] add ShapeVar representing non-neg
valued variable in a tensor shape
URL: https://github.com/apache/incubator-tvm/pull/4684#issuecomment-574323153
How about `TIndexVar` stands for tensor index variable?
---
tqchen opened a new pull request #4705: [REFACTOR][IR] Polish ir/type
URL: https://github.com/apache/incubator-tvm/pull/4705
- Use consistent constructor style to construct objects.
- Move env_func to ir as it is mainly used to construct IRs.
- Make docs consistent.
cc @zhiics @wwei
tqchen commented on a change in pull request #4684: [Arith] add ShapeVar
representing non-neg valued variable in a tensor shape
URL: https://github.com/apache/incubator-tvm/pull/4684#discussion_r366473075
##
File path: include/tvm/expr.h
##
@@ -115,6 +117,45 @@ class Var :
tqchen commented on issue #4684: [Arith] add ShapeVar representing non-neg
valued variable in a tensor shape
URL: https://github.com/apache/incubator-tvm/pull/4684#issuecomment-574285396
I don't have a better idea for name. Indeed shape could indicate a tuple
rather than an integer.
tqchen merged pull request #4702: [REFACTOR][IR] Initialize Unified IR Pass
Infra
URL: https://github.com/apache/incubator-tvm/pull/4702
This is an automated message from the Apache Git Service.
To respond to the message, pl
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 edc3674 [REFACTOR][IR] Move error.h into ir (#4701)
add d7d2a9b [REFACTOR][IR] Initialize Unified IR Pa
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_r366438454
##
File path: tests/python/frontend/tflite/test_forward.py
##
@@ -1113,6 +11
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_r366410943
##
File path: tests/python/frontend/tflite/test_forward.py
##
@@ -1113,6 +11
inadob commented on issue #4704: [Relay][Frontend][TFLite] Add parser support
for arg_min_max
URL: https://github.com/apache/incubator-tvm/pull/4704#issuecomment-574223969
can you please review this PR @anijain2305 @FrozenGene
---
inadob opened a new pull request #4704: [Relay][Frontend][TFLite] Add parser
support for arg_min_max
URL: https://github.com/apache/incubator-tvm/pull/4704
* this implementation supports only the case when the axis is a scalar
* tflite 1.13 removes all dims of size 1, Relay doesn't do th
masahi commented on issue #4497: [WIP] [Relay] Add a PyTorch to Relay Parser
URL: https://github.com/apache/incubator-tvm/pull/4497#issuecomment-574076780
@alexwong I tried your PR locally. With pytorch v1.3 it works, but they
introduced big change in
[#28408](https://github.com/pytorch/py
73 matches
Mail list logo