PhilippvK commented on issue #8953:
URL: https://github.com/apache/tvm/issues/8953#issuecomment-940589338
> So it seems like the "correct" thing to do is to define
TVM_CRT_MAX_STRLEN_PARAM_NAME, replace the current usages of
TVM_CRT_MAX_STRLEN_FUNCTION_NAME with TVM_CRT_MAX_STRLEN_PARAM_NA
AndrewZhaoLuo commented on a change in pull request #9184:
URL: https://github.com/apache/tvm/pull/9184#discussion_r726673423
##
File path: python/tvm/topi/searchsorted.py
##
@@ -0,0 +1,121 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor
This is an automated email from the ASF dual-hosted git repository.
areusch pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/tvm.git.
from 8725eb5 add stage to log (#9249)
add a7cf317 [TIR][USMP] Add a parallel to serial for loop converter pass
(#8469)
areusch merged pull request #8469:
URL: https://github.com/apache/tvm/pull/8469
--
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 the specific comment.
To unsubscribe, e-mail: commits-unsubscr..
areusch commented on a change in pull request #8469:
URL: https://github.com/apache/tvm/pull/8469#discussion_r726672928
##
File path: tests/python/unittest/test_tir_transform_convert_for_loops_serial.py
##
@@ -0,0 +1,62 @@
+# Licensed to the Apache Software Foundation (ASF) und
huajsj commented on a change in pull request #9108:
URL: https://github.com/apache/tvm/pull/9108#discussion_r726674300
##
File path: src/runtime/pipeline/pipeline_struct.h
##
@@ -0,0 +1,182 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contri
areusch commented on issue #8953:
URL: https://github.com/apache/tvm/issues/8953#issuecomment-940548495
@PhilippvK you are right, i think I see what happened here. #8380 was
authored a long time ago and sync'd up several times, and i think as part of a
"cleanup" these constants were change
This is an automated email from the ASF dual-hosted git repository.
wuwei pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/tvm.git.
from 01744d1 Fix typo in error message in CMakeLists.txt (#9251)
add 8725eb5 add stage to log (#9249)
No new revisions w
This is an automated email from the ASF dual-hosted git repository.
wuwei pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/tvm.git.
from f08dca8 [RPC] Fix Server connecting to RPC Tracker through a Proxy
(#9210)
add 01744d1 Fix typo in error message in
vinx13 merged pull request #9249:
URL: https://github.com/apache/tvm/pull/9249
--
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 the specific comment.
To unsubscribe, e-mail: commits-unsubscr...
vinx13 merged pull request #9251:
URL: https://github.com/apache/tvm/pull/9251
--
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 the specific comment.
To unsubscribe, e-mail: commits-unsubscr...
masahi commented on pull request #9108:
URL: https://github.com/apache/tvm/pull/9108#issuecomment-940527628
Your latest commit
https://github.com/apache/tvm/pull/9108/commits/cdfa5c221acedbaef248d7bc350e7505fe5d98e1
is full of grammar issues. Please take a look again.
--
This is an auto
masahi commented on a change in pull request #9108:
URL: https://github.com/apache/tvm/pull/9108#discussion_r726663545
##
File path: src/runtime/pipeline/pipeline_struct.h
##
@@ -0,0 +1,182 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contri
huajsj commented on a change in pull request #9108:
URL: https://github.com/apache/tvm/pull/9108#discussion_r726658113
##
File path: src/runtime/pipeline/pipeline_struct.h
##
@@ -0,0 +1,182 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contri
areusch commented on issue #9174:
URL: https://github.com/apache/tvm/issues/9174#issuecomment-940500616
@lhez please feel free to re-open if this is not fixed.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL a
areusch closed issue #9174:
URL: https://github.com/apache/tvm/issues/9174
--
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 the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@tvm
areusch edited a comment on issue #9239:
URL: https://github.com/apache/tvm/issues/9239#issuecomment-940500186
@WenguoLi could you provide a bit more detail please? May be related to #9208
--
This is an automated message from the Apache Git Service.
To respond to the message, please log o
areusch commented on issue #9239:
URL: https://github.com/apache/tvm/issues/9239#issuecomment-940500186
@WenguoLi could you provide a bit more detail please? May be related to #8809
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Gi
comaniac commented on a change in pull request #9248:
URL: https://github.com/apache/tvm/pull/9248#discussion_r726611923
##
File path: python/tvm/autotvm/graph_tuner/utils/traverse_graph.py
##
@@ -50,6 +51,8 @@ def expr2graph(expr, target_ops, node_dict, node_list):
Ea
areusch commented on pull request #8922:
URL: https://github.com/apache/tvm/pull/8922#issuecomment-940496946
@grant-arm just to follow-up on my comment, to use the Project API we
essentially need to write a `microtvm_api_server.py` which can use this tree as
a template in combination with
comaniac commented on a change in pull request #9248:
URL: https://github.com/apache/tvm/pull/9248#discussion_r726611467
##
File path: python/tvm/autotvm/graph_tuner/utils/traverse_graph.py
##
@@ -130,8 +130,11 @@ def _traverse_expr(node):
call = relay.Call(nod
rkimball commented on a change in pull request #9248:
URL: https://github.com/apache/tvm/pull/9248#discussion_r726609974
##
File path: python/tvm/autotvm/graph_tuner/utils/traverse_graph.py
##
@@ -130,8 +130,11 @@ def _traverse_expr(node):
call = relay.Call(nod
comaniac commented on a change in pull request #9248:
URL: https://github.com/apache/tvm/pull/9248#discussion_r726605124
##
File path: python/tvm/autotvm/graph_tuner/utils/traverse_graph.py
##
@@ -130,8 +130,11 @@ def _traverse_expr(node):
call = relay.Call(nod
comaniac commented on a change in pull request #9248:
URL: https://github.com/apache/tvm/pull/9248#discussion_r726605124
##
File path: python/tvm/autotvm/graph_tuner/utils/traverse_graph.py
##
@@ -130,8 +130,11 @@ def _traverse_expr(node):
call = relay.Call(nod
AndrewZhaoLuo opened a new pull request #9252:
URL: https://github.com/apache/tvm/pull/9252
WIP until I profile and think of a better way to pass these strings in. E.g.
through target strings or something seperate.
Link #9223
--
This is an automated message from the Apache Git S
rkimball commented on a change in pull request #9248:
URL: https://github.com/apache/tvm/pull/9248#discussion_r726566056
##
File path: python/tvm/autotvm/graph_tuner/utils/traverse_graph.py
##
@@ -130,8 +130,11 @@ def _traverse_expr(node):
call = relay.Call(nod
rkimball commented on a change in pull request #9248:
URL: https://github.com/apache/tvm/pull/9248#discussion_r726566056
##
File path: python/tvm/autotvm/graph_tuner/utils/traverse_graph.py
##
@@ -130,8 +130,11 @@ def _traverse_expr(node):
call = relay.Call(nod
comaniac commented on a change in pull request #9248:
URL: https://github.com/apache/tvm/pull/9248#discussion_r726562287
##
File path: python/tvm/autotvm/graph_tuner/utils/traverse_graph.py
##
@@ -130,8 +130,11 @@ def _traverse_expr(node):
call = relay.Call(nod
rkimball commented on a change in pull request #9248:
URL: https://github.com/apache/tvm/pull/9248#discussion_r726550394
##
File path: python/tvm/autotvm/graph_tuner/utils/traverse_graph.py
##
@@ -130,8 +130,11 @@ def _traverse_expr(node):
call = relay.Call(nod
tqchen commented on pull request #18:
URL: https://github.com/apache/tvm-rfcs/pull/18#issuecomment-940411463
@kparzysz-quic this is right. On the other hand, we still on conversion of
DLDataType and DataType(e.g. getting DataType from ndarray) in many cases. So
ideally we want to keep them
sergey-grovety commented on a change in pull request #9233:
URL: https://github.com/apache/tvm/pull/9233#discussion_r726534192
##
File path: python/tvm/topi/arm_cpu/cortex_m7/micro_kernel/relu.py
##
@@ -0,0 +1,78 @@
+# Licensed to the Apache Software Foundation (ASF) under one
ccjoechou commented on pull request #9235:
URL: https://github.com/apache/tvm/pull/9235#issuecomment-940368033
Yes and I am trying to reproduce them locally in order to find resolutions.
* Joe
From: Wuwei Lin ***@***.***>
Sent: Monday, October 11, 2021 12:00 PM
To
vinx13 commented on pull request #9235:
URL: https://github.com/apache/tvm/pull/9235#issuecomment-940364473
There are some failing test cases on CI that relate to this change
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub an
This is an automated email from the ASF dual-hosted git repository.
tqchen pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/tvm.git.
from 95cde0c [AlterLayout] Strided slice layout transform fix (disallow
NCHW4c -> NCHW etc properly) (#9245)
add f08dca8
tqchen commented on pull request #9210:
URL: https://github.com/apache/tvm/pull/9210#issuecomment-940363624
Thank you @KJlaccHoeUM9l !
--
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 the specifi
tqchen merged pull request #9210:
URL: https://github.com/apache/tvm/pull/9210
--
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 the specific comment.
To unsubscribe, e-mail: commits-unsubscr...
comaniac commented on a change in pull request #9248:
URL: https://github.com/apache/tvm/pull/9248#discussion_r726491354
##
File path: python/tvm/autotvm/graph_tuner/utils/traverse_graph.py
##
@@ -130,8 +130,11 @@ def _traverse_expr(node):
call = relay.Call(nod
electriclilies opened a new pull request #9251:
URL: https://github.com/apache/tvm/pull/9251
There is a typo in one of the error messages in the CMakeLists.txt that
causes the script to fail if the error message is printed out. This PR fixes
the typo.
--
This is an automated message fr
rkimball commented on a change in pull request #9248:
URL: https://github.com/apache/tvm/pull/9248#discussion_r726475382
##
File path: python/tvm/autotvm/graph_tuner/utils/traverse_graph.py
##
@@ -129,7 +131,7 @@ def _traverse_expr(node):
mod = tvm.IRModule.fro
kparzysz-quic commented on pull request #18:
URL: https://github.com/apache/tvm-rfcs/pull/18#issuecomment-940244579
Right, but users of DLPack will not see `DataType`, only `DLDataType`,
unless they use TVM as well. Changing `DataType` will only affect those that
use `include/tvm/runtime/
mehrdadh commented on issue #9226:
URL: https://github.com/apache/tvm/issues/9226#issuecomment-940233309
and this one is from Image Classification model (one of the MLPerfTiny
models):
```
Check failed: (expr_equal(lhs, rhs)) is false: Failed to match the compute
with TensorIntri
FranckQC opened a new issue #9250:
URL: https://github.com/apache/tvm/issues/9250
Hi everyone,
Following the discussion started
[here](https://discuss.tvm.apache.org/t/tir-are-multiple-let-in-with-the-same-var-allowed-ssa/10881/6)
I would like to discuss a thing that seems weird in
mehrdadh commented on pull request #9249:
URL: https://github.com/apache/tvm/pull/9249#issuecomment-940231325
cc @areusch @vinx13
--
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 the specific co
rkimball commented on a change in pull request #9248:
URL: https://github.com/apache/tvm/pull/9248#discussion_r726459570
##
File path: python/tvm/autotvm/graph_tuner/utils/traverse_graph.py
##
@@ -129,7 +131,7 @@ def _traverse_expr(node):
mod = tvm.IRModule.fro
mehrdadh opened a new pull request #9249:
URL: https://github.com/apache/tvm/pull/9249
This will help to identify which stage (operator) has faile.
--
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
comaniac commented on a change in pull request #9248:
URL: https://github.com/apache/tvm/pull/9248#discussion_r726455535
##
File path: python/tvm/autotvm/graph_tuner/utils/traverse_graph.py
##
@@ -129,7 +131,7 @@ def _traverse_expr(node):
mod = tvm.IRModule.fro
This is an automated email from the ASF dual-hosted git repository.
wuwei pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/tvm.git.
from 5ad2f77 [Relay] Gather op dynamic input support (#9240)
add 95cde0c [AlterLayout] Strided slice layout transform fix
tqchen commented on pull request #18:
URL: https://github.com/apache/tvm-rfcs/pull/18#issuecomment-940227361
@kparzysz-quic DatatType is a thin wrapper around DLDataType
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
vinx13 closed issue #8759:
URL: https://github.com/apache/tvm/issues/8759
--
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 the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@tvm.
mehrdadh edited a comment on issue #9226:
URL: https://github.com/apache/tvm/issues/9226#issuecomment-940226266
@sergey-grovety thanks for working on it. I'm adding other instances that
this happens.
- from Visual Wake Word model:
```
Check failed: (expr_equal(lhs, rhs)) is fal
vinx13 merged pull request #9245:
URL: https://github.com/apache/tvm/pull/9245
--
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 the specific comment.
To unsubscribe, e-mail: commits-unsubscr...
rkimball opened a new pull request #9248:
URL: https://github.com/apache/tvm/pull/9248
This allows for targets other than llvm to run graph tuning.
--
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
mehrdadh commented on issue #9226:
URL: https://github.com/apache/tvm/issues/9226#issuecomment-940226266
@sergey-grovety thanks for working on it. I'm adding other instances that
this happens.
- from Visual Wake Word model:
```
E Check failed: (expr_equal(lhs, rhs))
kparzysz-quic commented on pull request #18:
URL: https://github.com/apache/tvm-rfcs/pull/18#issuecomment-940209842
> DataType and DLDataType are part of DLPack standard,
The `DataType` type is defined in TVM, it's not present in DLPack. Or am I
missing something?
--
This is an a
mehrdadh commented on a change in pull request #9233:
URL: https://github.com/apache/tvm/pull/9233#discussion_r726247929
##
File path: python/tvm/topi/arm_cpu/cortex_m7/micro_kernel/relu.py
##
@@ -0,0 +1,78 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or
comaniac commented on pull request #7658:
URL: https://github.com/apache/tvm/pull/7658#issuecomment-940174322
Gentle ping @akmaru please fix the CI
--
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
comaniac commented on pull request #8131:
URL: https://github.com/apache/tvm/pull/8131#issuecomment-940174007
Gentle ping @akmaru please resolve the conflict and pass the CI.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub an
This is an automated email from the ASF dual-hosted git repository.
comaniac pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/tvm.git
The following commit(s) were added to refs/heads/main by this push:
new 5ad2f77 [Relay] Gather op dynamic input support (#92
comaniac merged pull request #9240:
URL: https://github.com/apache/tvm/pull/9240
--
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 the specific comment.
To unsubscribe, e-mail: commits-unsubscr.
comaniac commented on pull request #9240:
URL: https://github.com/apache/tvm/pull/9240#issuecomment-940173392
Thanks @masahi
--
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 the specific comment
kparzysz-quic commented on a change in pull request #18:
URL: https://github.com/apache/tvm-rfcs/pull/18#discussion_r726235647
##
File path: 0018-initial-sve-addition.md
##
@@ -0,0 +1,229 @@
+- Feature Name: Adding Initial SVE Support to TVM
+- Start Date: 2021-07-30
+- RFC PR
ashutosh-arm commented on pull request #9247:
URL: https://github.com/apache/tvm/pull/9247#issuecomment-940153989
EthosU and CMSIS-NN tests passed locally for me with the new version.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
ashutosh-arm opened a new pull request #9247:
URL: https://github.com/apache/tvm/pull/9247
Need to bump up CMSIS version in the docker image to match TFLu.
CC: @Mousius @grant-arm @areusch @leandron
--
This is an automated message from the Apache Git Service.
To respond to the
This is an automated email from the ASF dual-hosted git repository.
mbaret pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/tvm.git
The following commit(s) were added to refs/heads/main by this push:
new 8ba0451 Arm(R) Ethos(TM)-U NPU Depthwise2d operator su
mbaret commented on pull request #9209:
URL: https://github.com/apache/tvm/pull/9209#issuecomment-940139122
This is now merged, thanks everyone!
--
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 t
mbaret merged pull request #9209:
URL: https://github.com/apache/tvm/pull/9209
--
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 the specific comment.
To unsubscribe, e-mail: commits-unsubscr...
tmoreau89 commented on issue #8838:
URL: https://github.com/apache/tvm/issues/8838#issuecomment-940100968
Thank you I somehow failed to catch it, I'm adding
https://github.com/apache/tvm/pull/9063 to the table, thank you for your
contributions.
--
This is an automated message from the A
gayatripk1 commented on issue #8838:
URL: https://github.com/apache/tvm/issues/8838#issuecomment-940096572
> @WenheLI and @gayatripk1, is support for `MatMulInteger` and
`com.microsoft.QLinearLeakyRelu` still on your plates? Just checking in
periodically.
com.microsoft.QLinearLeakyR
tmoreau89 commented on issue #8838:
URL: https://github.com/apache/tvm/issues/8838#issuecomment-940093723
@WenheLI and @gayatripk1, is support for `MatMulInteger` and
`com.microsoft.QLinearLeakyRelu` still on your plates? Just checking in
periodically.
--
This is an automated message fr
tmoreau89 commented on issue #8838:
URL: https://github.com/apache/tvm/issues/8838#issuecomment-940092842
QLinearMatMul support is now merged! Thank you @cconvey for completing this
task
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on
tmoreau89 commented on pull request #8952:
URL: https://github.com/apache/tvm/pull/8952#issuecomment-940064700
Thank you @cconvey @anwang2009 @AndrewZhaoLuo @mbrookhart the PR has been
merged!
--
This is an automated message from the Apache Git Service.
To respond to the message, please
This is an automated email from the ASF dual-hosted git repository.
moreau pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/tvm.git
The following commit(s) were added to refs/heads/main by this push:
new 0fa860c [TVM] Add importer for ONNX QLinearMatMul op (
tmoreau89 merged pull request #8952:
URL: https://github.com/apache/tvm/pull/8952
--
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 the specific comment.
To unsubscribe, e-mail: commits-unsubscr
mbaret commented on pull request #9190:
URL: https://github.com/apache/tvm/pull/9190#issuecomment-940046316
Thanks @Mousius, @mbs-octoml this is merged
--
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
This is an automated email from the ASF dual-hosted git repository.
mbaret pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/tvm.git.
from 74aa856 [DOCS] Fix installation from source link some text (#9238)
add 0b91e53 Initial Implementation of TIRToRunti
mbaret merged pull request #9190:
URL: https://github.com/apache/tvm/pull/9190
--
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 the specific comment.
To unsubscribe, e-mail: commits-unsubscr...
Mousius opened a new pull request #9246:
URL: https://github.com/apache/tvm/pull/9246
This is the underpinning Executor/Runtime objects for
https://github.com/apache/tvm-rfcs/pull/29, this doesn't change the wiring as
yet since that's a pretty big change in itself. Most of this patch is TV
Mousius commented on pull request #9190:
URL: https://github.com/apache/tvm/pull/9190#issuecomment-939995919
@mbaret could you take a look, @mbs-octoml seems happy so I'm happy
:smile_cat: will look to follow up in a later PR on the minor nit.
--
This is an automated message from the Ap
PhilippvK commented on pull request #9243:
URL: https://github.com/apache/tvm/pull/9243#issuecomment-939977914
Ethosu and CMSIS-NN related builds seem to fail in CI, I will look into this
and follow up with a fix for that.
--
This is an automated message from the Apache Git Service.
To r
Mousius commented on a change in pull request #28:
URL: https://github.com/apache/tvm-rfcs/pull/28#discussion_r726013981
##
File path: rfcs/0028-command-line-registry-composition.md
##
@@ -0,0 +1,153 @@
+- Feature Name: Command Line Composition from Internal Registry
+- Start D
masahi commented on a change in pull request #9108:
URL: https://github.com/apache/tvm/pull/9108#discussion_r725965693
##
File path: python/tvm/contrib/pipeline_executor.py
##
@@ -139,12 +187,12 @@ def get_owner_idx(self):
if isinstance(self.io_owner, PipelineConfi
lhutton1 opened a new pull request #9244:
URL: https://github.com/apache/tvm/pull/9244
Currently a conv2d pattern is matched when no bias is present. However,
legalization expects a bias to be present, therefore causing an error when this
is not the case. For now, enforce a bias when offlo
PhilippvK opened a new pull request #9243:
URL: https://github.com/apache/tvm/pull/9243
## Motivation
I ran into the issue that I need to use C++ features in a BYOC generated
kernel which is tricky to achieve because those are typically exported as `.c`
files.
I am aware of the p
83 matches
Mail list logo