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
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
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
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
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
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
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
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
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.
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()
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
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
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
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
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
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
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
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
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
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
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
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
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 {
}
}
+
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)
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
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
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
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
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,
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
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
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
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
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
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)
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
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
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
+ *
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
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
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
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,
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
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
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
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
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 =
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
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
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
+ *
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
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
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 =>
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
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
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
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
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.
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?
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
+
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
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
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
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
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
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
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
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.
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
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
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
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.
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.
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
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
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
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
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
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
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
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
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
82 matches
Mail list logo