manupa-arm commented on a change in pull request #7266:
URL: https://github.com/apache/tvm/pull/7266#discussion_r561233084
##
File path: src/runtime/crt/host/main.cc
##
@@ -93,6 +94,20 @@ tvm_crt_error_t TVMPlatformTimerStop(double*
elapsed_time_seconds) {
manupa-arm commented on a change in pull request #7266:
URL: https://github.com/apache/tvm/pull/7266#discussion_r561233084
##
File path: src/runtime/crt/host/main.cc
##
@@ -93,6 +94,20 @@ tvm_crt_error_t TVMPlatformTimerStop(double*
elapsed_time_seconds) {
jchia opened a new pull request #7320:
URL: https://github.com/apache/tvm/pull/7320
IsNan was added to tensorflow.rst in fa1b859f but this commit makes IsNan
actually work
This is an automated message from the Apache
t-vi commented on pull request #7231:
URL: https://github.com/apache/tvm/pull/7231#issuecomment-764527672
To keep things transparent, I'm copying a terse bit of explanation from
slack:
To see what goes wrong and why the test doesn't catch it, compare the graph
for traced and
manupa-arm commented on a change in pull request #7266:
URL: https://github.com/apache/tvm/pull/7266#discussion_r561233084
##
File path: src/runtime/crt/host/main.cc
##
@@ -93,6 +94,20 @@ tvm_crt_error_t TVMPlatformTimerStop(double*
elapsed_time_seconds) {
This is an automated email from the ASF dual-hosted git repository.
masahi pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/tvm.git.
from f829403 [TOPI] Rewrite GPU argwhere using exclusive scan (#7314)
add 20e03bc [COMMUNITY] @jwfromm -> Committer
masahi merged pull request #7316:
URL: https://github.com/apache/tvm/pull/7316
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
jchia edited a comment on pull request #7320:
URL: https://github.com/apache/tvm/pull/7320#issuecomment-764676518
> Thanks for the patch. Can you please add a test case for `IsNan`, to make
sure it doesn't regress in future?
I'm not familiar with the code base, but I think the test
leandron commented on pull request #7320:
URL: https://github.com/apache/tvm/pull/7320#issuecomment-764707942
> > Thanks for the patch. Can you please add a test case for `IsNan`, to
make sure it doesn't regress in future?
>
> I'm not familiar with the code base, but I think the
jchia edited a comment on pull request #7320:
URL: https://github.com/apache/tvm/pull/7320#issuecomment-764676518
> Thanks for the patch. Can you please add a test case for `IsNan`, to make
sure it doesn't regress in future?
I'm not familiar with the code base, but I think the test
jchia commented on pull request #7320:
URL: https://github.com/apache/tvm/pull/7320#issuecomment-764676518
> Thanks for the patch. Can you please add a test case for `IsNan`, to make
sure it doesn't regress in future?
I'm not familiar with the code base. Where can I add this test? I
This is an automated email from the ASF dual-hosted git repository.
tqchen 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 8524b28 [µTVM] Add TVMPlatformGenerateRandom, a
tqchen merged pull request #7266:
URL: https://github.com/apache/tvm/pull/7266
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
leandron commented on pull request #7320:
URL: https://github.com/apache/tvm/pull/7320#issuecomment-764674422
Thanks for the patch. Can you please add a test case for `IsNan`, to make
sure it doesn't regress in future?
This
hzfan opened a new pull request #7321:
URL: https://github.com/apache/tvm/pull/7321
`te.gradient` may generate different (but equivalent) backward compute for a
single forward, which may result in Ansor miss. This pr makes sure
`te.gradient` always generates the same backward.
cc
tkonolige commented on a change in pull request #7287:
URL: https://github.com/apache/tvm/pull/7287#discussion_r562038095
##
File path: python/tvm/topi/random/kernel.py
##
@@ -234,6 +242,18 @@ def gen_ir(gen_ptr, out_gen_ptr, out_array_ptr):
out_gen =
SebastianBoblestETAS opened a new issue #7322:
URL: https://github.com/apache/tvm/issues/7322
The options print_top and outputs are currently incompatible,
get_top_results should work on a copy of outputs.
Otherwise sorted outputs are stored in .npz
tkonolige commented on a change in pull request #7308:
URL: https://github.com/apache/tvm/pull/7308#discussion_r562019758
##
File path: python/tvm/topi/cuda/sparse.py
##
@@ -295,7 +295,14 @@ def is_valid_for_sparse_dense_padded(data, weight_data):
"""
#
comaniac commented on issue #7322:
URL: https://github.com/apache/tvm/issues/7322#issuecomment-764806994
Thanks for pointing out. Would you send a PR to fix it?
Also cc @leandron
This is an automated message from the
This is an automated email from the ASF dual-hosted git repository.
jwfromm 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 17ae44d add a shape function and dynamic test for
jwfromm merged pull request #7324:
URL: https://github.com/apache/tvm/pull/7324
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
slyubomirsky commented on pull request #7323:
URL: https://github.com/apache/tvm/pull/7323#issuecomment-765089486
I've also been bitten by `TupleWrapper`s lol, maybe it shouldn't be the
default, since it's only intended as a convenience when working with the Python
APIs.
comaniac commented on a change in pull request #7313:
URL: https://github.com/apache/tvm/pull/7313#discussion_r561459545
##
File path: tutorials/auto_scheduler/tune_sparse_x86.py
##
@@ -0,0 +1,331 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more
tqchen commented on a change in pull request #7152:
URL: https://github.com/apache/tvm/pull/7152#discussion_r562270981
##
File path: include/tvm/runtime/packed_func.h
##
@@ -1377,7 +1449,7 @@ inline TObjectRef TVMPODValue_::AsObjectRef() const {
using ContainerType =
liangfu commented on a change in pull request #7286:
URL: https://github.com/apache/tvm/pull/7286#discussion_r562358753
##
File path: tests/python/contrib/test_verilator/infrastructure.py
##
@@ -66,10 +68,43 @@ def offload(mod):
return mod
+def verilator_app_path():
vegaluisjose commented on a change in pull request #7286:
URL: https://github.com/apache/tvm/pull/7286#discussion_r562375277
##
File path: tests/python/contrib/test_verilator/infrastructure.py
##
@@ -66,10 +68,43 @@ def offload(mod):
return mod
+def
mbrookhart commented on pull request #7295:
URL: https://github.com/apache/tvm/pull/7295#issuecomment-764917665
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
masahi merged pull request #7325:
URL: https://github.com/apache/tvm/pull/7325
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
tmoreau89 opened a new pull request #7326:
URL: https://github.com/apache/tvm/pull/7326
This tutorial adapts the x86 autoscheduler tutorial to work over RPC and
target aarch64-based Rapsberry Pi.
This is an automated
tmoreau89 merged pull request #7295:
URL: https://github.com/apache/tvm/pull/7295
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
comaniac commented on a change in pull request #7321:
URL: https://github.com/apache/tvm/pull/7321#discussion_r562115763
##
File path: tests/python/unittest/test_te_autodiff.py
##
@@ -343,7 +343,23 @@ def test_reduction_init():
check_grad(B, A0)
+def test_stable():
+
tkonolige commented on a change in pull request #7308:
URL: https://github.com/apache/tvm/pull/7308#discussion_r562019758
##
File path: python/tvm/topi/cuda/sparse.py
##
@@ -295,7 +295,14 @@ def is_valid_for_sparse_dense_padded(data, weight_data):
"""
#
icemelon9 commented on a change in pull request #7316:
URL: https://github.com/apache/tvm/pull/7316#discussion_r561357994
##
File path: CONTRIBUTORS.md
##
@@ -47,6 +47,7 @@ We do encourage everyone to work anything they are interested
in.
- [Wei
codeislife99 commented on pull request #7231:
URL: https://github.com/apache/tvm/pull/7231#issuecomment-764424330
```
class Unsqueeze3(Module):
def forward(self, *args):
_ = args[0].unsqueeze_(2)
y = args[0] * 2
return y
```
altanh commented on pull request #7323:
URL: https://github.com/apache/tvm/pull/7323#issuecomment-765102893
> I've also been bitten by `TupleWrapper`s lol, maybe it shouldn't be the
default, since it's only intended as a convenience when working with the Python
APIs.
Indeed, it
icemelon9 merged pull request #7311:
URL: https://github.com/apache/tvm/pull/7311
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
tkonolige commented on a change in pull request #7267:
URL: https://github.com/apache/tvm/pull/7267#discussion_r562220135
##
File path: python/tvm/relay/frontend/tensorflow.py
##
@@ -942,7 +942,10 @@ def _impl(inputs, attr, params, mod):
)
if sparse_lhs:
-
liangfu commented on a change in pull request #7286:
URL: https://github.com/apache/tvm/pull/7286#discussion_r562358753
##
File path: tests/python/contrib/test_verilator/infrastructure.py
##
@@ -66,10 +68,43 @@ def offload(mod):
return mod
+def verilator_app_path():
masahi merged pull request #7325:
URL: https://github.com/apache/tvm/pull/7325
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
This is an automated email from the ASF dual-hosted git repository.
masahi 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 790344c relax tolerance for dlpack test (#7325)
masahi closed issue #7312:
URL: https://github.com/apache/tvm/issues/7312
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
altanh commented on pull request #7323:
URL: https://github.com/apache/tvm/pull/7323#issuecomment-765102893
> I've also been bitten by `TupleWrapper`s lol, maybe it shouldn't be the
default, since it's only intended as a convenience when working with the Python
APIs.
Indeed, it
t-vi commented on pull request #7231:
URL: https://github.com/apache/tvm/pull/7231#issuecomment-764446329
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
ZihengJiang commented on a change in pull request #6370:
URL: https://github.com/apache/tvm/pull/6370#discussion_r562280157
##
File path: python/tvm/topi/einsum.py
##
@@ -0,0 +1,46 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license
tkonolige commented on a change in pull request #7287:
URL: https://github.com/apache/tvm/pull/7287#discussion_r562038095
##
File path: python/tvm/topi/random/kernel.py
##
@@ -234,6 +242,18 @@ def gen_ir(gen_ptr, out_gen_ptr, out_array_ptr):
out_gen =
vegaluisjose commented on a change in pull request #7286:
URL: https://github.com/apache/tvm/pull/7286#discussion_r562375277
##
File path: tests/python/contrib/test_verilator/infrastructure.py
##
@@ -66,10 +68,43 @@ def offload(mod):
return mod
+def
masahi closed issue #7312:
URL: https://github.com/apache/tvm/issues/7312
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
areusch commented on a change in pull request #7152:
URL: https://github.com/apache/tvm/pull/7152#discussion_r56226
##
File path: include/tvm/runtime/packed_func.h
##
@@ -1377,7 +1449,7 @@ inline TObjectRef TVMPODValue_::AsObjectRef() const {
using ContainerType =
tqchen merged pull request #7266:
URL: https://github.com/apache/tvm/pull/7266
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
mbrookhart commented on pull request #7314:
URL: https://github.com/apache/tvm/pull/7314#issuecomment-764001126
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
tkonolige commented on a change in pull request #7267:
URL: https://github.com/apache/tvm/pull/7267#discussion_r562220135
##
File path: python/tvm/relay/frontend/tensorflow.py
##
@@ -942,7 +942,10 @@ def _impl(inputs, attr, params, mod):
)
if sparse_lhs:
-
tqchen commented on a change in pull request #7152:
URL: https://github.com/apache/tvm/pull/7152#discussion_r562270981
##
File path: include/tvm/runtime/packed_func.h
##
@@ -1377,7 +1449,7 @@ inline TObjectRef TVMPODValue_::AsObjectRef() const {
using ContainerType =
tqchen commented on a change in pull request #7152:
URL: https://github.com/apache/tvm/pull/7152#discussion_r562281721
##
File path: include/tvm/runtime/packed_func.h
##
@@ -229,13 +229,13 @@ class TypedPackedFunc {
* \endcode
*
* \param typed_lambda typed lambda
mbrookhart commented on pull request #7295:
URL: https://github.com/apache/tvm/pull/7295#issuecomment-765001735
Yes, my originally attempt failed a bunch prelude/adt tests because I didn't
support that :)
This is an
ZihengJiang commented on a change in pull request #6370:
URL: https://github.com/apache/tvm/pull/6370#discussion_r562280157
##
File path: python/tvm/topi/einsum.py
##
@@ -0,0 +1,46 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license
tkonolige commented on pull request #7152:
URL: https://github.com/apache/tvm/pull/7152#issuecomment-764964882
@tqchen @areusch @junrushao1994 I think this PR is in a good state to merge.
Could you review again?
This is an
areusch commented on a change in pull request #7152:
URL: https://github.com/apache/tvm/pull/7152#discussion_r56226
##
File path: include/tvm/runtime/packed_func.h
##
@@ -1377,7 +1449,7 @@ inline TObjectRef TVMPODValue_::AsObjectRef() const {
using ContainerType =
mbrookhart opened a new pull request #7324:
URL: https://github.com/apache/tvm/pull/7324
@jwfromm
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
mbrookhart commented on pull request #7295:
URL: https://github.com/apache/tvm/pull/7295#issuecomment-764917665
@jroesch @tmoreau89 Hit a driver bug in a CI node last night than Tianqi
fixed, otherwise this passed tests, I think it's ready for review while the
rebase runs.
masahi opened a new pull request #7325:
URL: https://github.com/apache/tvm/pull/7325
https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/PR-7303/12/pipeline/
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 main
in repository https://gitbox.apache.org/repos/asf/tvm.git.
from 8524b28 [µTVM] Add TVMPlatformGenerateRandom, a non-cryptographic
random number generator. (#7266)
add fc9e264
tqchen merged pull request #7320:
URL: https://github.com/apache/tvm/pull/7320
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
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 7b6a1a7 Fix an issue with dynamic functions
tmoreau89 commented on pull request #7295:
URL: https://github.com/apache/tvm/pull/7295#issuecomment-765022117
Thank you @mbrookhart and @altanh the PR has been merged.
This is an automated message from the Apache Git
tmoreau89 merged pull request #7295:
URL: https://github.com/apache/tvm/pull/7295
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-vi edited a comment on pull request #7231:
URL: https://github.com/apache/tvm/pull/7231#issuecomment-756764112
I don't want to ruin the party, but does `unsqueeze_` work as is?
We would want to update future use of the input to refer to the output (same
for any of the "simple"
luyaor commented on issue #7262:
URL: https://github.com/apache/tvm/issues/7262#issuecomment-765198567
@jwfromm
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
tqchen commented on issue #7319:
URL: https://github.com/apache/tvm/issues/7319#issuecomment-764896482
thanks @ggardet can you send a new PR ?
This is an automated message from the Apache Git Service.
To respond to the
FrozenGene merged pull request #6998:
URL: https://github.com/apache/tvm/pull/6998
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
d-smirnov commented on pull request #7114:
URL: https://github.com/apache/tvm/pull/7114#issuecomment-763957745
@FrozenGene please, take a look
This is an automated message from the Apache Git Service.
To respond to the
masahi merged pull request #7316:
URL: https://github.com/apache/tvm/pull/7316
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
jchia edited a comment on pull request #7320:
URL: https://github.com/apache/tvm/pull/7320#issuecomment-764676518
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
masahi commented on pull request #7316:
URL: https://github.com/apache/tvm/pull/7316#issuecomment-764154479
@icemelon9 please kick again
This is an automated message from the Apache Git Service.
To respond to the message,
tkonolige commented on pull request #7152:
URL: https://github.com/apache/tvm/pull/7152#issuecomment-764964882
@tqchen @areusch @junrushao1994 I think this PR is in a good state to merge.
Could you review again?
This is an
monklof commented on pull request #6898:
URL: https://github.com/apache/tvm/pull/6898#issuecomment-765199446
> @monklof any updates? Thanks in advance!
updated, check the lastest commit
This is an automated message
slyubomirsky commented on pull request #7323:
URL: https://github.com/apache/tvm/pull/7323#issuecomment-765089486
I've also been bitten by `TupleWrapper`s lol, maybe it shouldn't be the
default, since it's only intended as a convenience when working with the Python
APIs.
tqchen merged pull request #7320:
URL: https://github.com/apache/tvm/pull/7320
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
jchia commented on pull request #7320:
URL: https://github.com/apache/tvm/pull/7320#issuecomment-764676518
> Thanks for the patch. Can you please add a test case for `IsNan`, to make
sure it doesn't regress in future?
I'm not familiar with the code base. Where can I add this test? I
masahi commented on pull request #7314:
URL: https://github.com/apache/tvm/pull/7314#issuecomment-764009115
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
manupa-arm commented on a change in pull request #7266:
URL: https://github.com/apache/tvm/pull/7266#discussion_r561233084
##
File path: src/runtime/crt/host/main.cc
##
@@ -93,6 +94,20 @@ tvm_crt_error_t TVMPlatformTimerStop(double*
elapsed_time_seconds) {
leandron commented on pull request #7320:
URL: https://github.com/apache/tvm/pull/7320#issuecomment-764674422
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
comaniac commented on issue #7322:
URL: https://github.com/apache/tvm/issues/7322#issuecomment-764806994
Thanks for pointing out. Would you send a PR to fix it?
Also cc @leandron
This is an automated message from the
tqchen commented on a change in pull request #7316:
URL: https://github.com/apache/tvm/pull/7316#discussion_r561346329
##
File path: CONTRIBUTORS.md
##
@@ -47,6 +47,7 @@ We do encourage everyone to work anything they are interested
in.
- [Wei Chen](https://github.com/wweic):
monklof commented on a change in pull request #6898:
URL: https://github.com/apache/tvm/pull/6898#discussion_r562437313
##
File path: python/tvm/topi/cuda/dense.py
##
@@ -77,8 +77,8 @@ def _callback(op):
def _schedule_dense_small_batch(cfg, s, C):
-A, _ =
tmoreau89 commented on pull request #7295:
URL: https://github.com/apache/tvm/pull/7295#issuecomment-765022117
Thank you @mbrookhart and @altanh the PR has been merged.
This is an automated message from the Apache Git
masahi merged pull request #7314:
URL: https://github.com/apache/tvm/pull/7314
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
luyaor commented on issue #7270:
URL: https://github.com/apache/tvm/issues/7270#issuecomment-765198018
@jwfromm
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
zhanghaohit commented on pull request #6126:
URL: https://github.com/apache/tvm/pull/6126#issuecomment-764175758
@tqchen seems CI is hang? Could you take a look at it? Thanks.
This is an automated message from the Apache Git
jwfromm merged pull request #7324:
URL: https://github.com/apache/tvm/pull/7324
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
tqchen commented on issue #7312:
URL: https://github.com/apache/tvm/issues/7312#issuecomment-764898503
i see, @masahi can you send a PR to fix it?
This is an automated message from the Apache Git Service.
To respond to the
tqchen commented on issue #7319:
URL: https://github.com/apache/tvm/issues/7319#issuecomment-764896482
thanks @ggardet can you send a new PR ?
This is an automated message from the Apache Git Service.
To respond to the
tqchen commented on issue #7312:
URL: https://github.com/apache/tvm/issues/7312#issuecomment-764898503
i see, @masahi can you send a PR to fix it?
This is an automated message from the Apache Git Service.
To respond to the
comaniac commented on a change in pull request #7321:
URL: https://github.com/apache/tvm/pull/7321#discussion_r562115763
##
File path: tests/python/unittest/test_te_autodiff.py
##
@@ -343,7 +343,23 @@ def test_reduction_init():
check_grad(B, A0)
+def test_stable():
+
altanh opened a new pull request #7323:
URL: https://github.com/apache/tvm/pull/7323
Added gradients for:
- `concatenate`
- `reshape_like`
- `where`
- `less_equal`
cc @antinucleon @MarisaKirisame
Somewhat tangential: the behavior for `split` in Python Relay is
94 matches
Mail list logo