[GitHub] [tvm] delldu commented on a change in pull request #8443: Support aten::grid_sampler, aten::im2col, Fix bug aten::Float

2021-07-13 Thread GitBox
delldu commented on a change in pull request #8443: URL: https://github.com/apache/tvm/pull/8443#discussion_r669321288 ## File path: tests/python/frontend/pytorch/test_forward.py ## @@ -3912,6 +3912,109 @@ def forward(self, x): verify_model(Flip(axis=-1), input_data=input)

[GitHub] [tvm] masahi commented on pull request #8443: Support aten::grid_sampler, aten::im2col, Fix bug aten::Float

2021-07-13 Thread GitBox
masahi commented on pull request #8443: URL: https://github.com/apache/tvm/pull/8443#issuecomment-879608301 @delldu I don't see a need to support scripting for these ops, so can you simplify this PR? You don't need to test with `verify_script_model`. -- This is an automated message from

[GitHub] [tvm] masahi commented on a change in pull request #8443: Support aten::grid_sampler, aten::im2col, Fix bug aten::Float

2021-07-13 Thread GitBox
masahi commented on a change in pull request #8443: URL: https://github.com/apache/tvm/pull/8443#discussion_r669303917 ## File path: python/tvm/relay/frontend/pytorch.py ## @@ -1875,7 +1875,9 @@ def Bool(self, inputs, input_types): def Float(self, inputs, input_types):

[GitHub] [tvm] masahi commented on a change in pull request #8443: Support aten::grid_sampler, aten::im2col, Fix bug aten::Float

2021-07-13 Thread GitBox
masahi commented on a change in pull request #8443: URL: https://github.com/apache/tvm/pull/8443#discussion_r669301896 ## File path: tests/python/frontend/pytorch/test_forward.py ## @@ -3912,6 +3912,109 @@ def forward(self, x): verify_model(Flip(axis=-1), input_data=input)

[GitHub] [tvm] masahi commented on a change in pull request #8443: Support aten::grid_sampler, aten::im2col, Fix bug aten::Float

2021-07-13 Thread GitBox
masahi commented on a change in pull request #8443: URL: https://github.com/apache/tvm/pull/8443#discussion_r669301896 ## File path: tests/python/frontend/pytorch/test_forward.py ## @@ -3912,6 +3912,109 @@ def forward(self, x): verify_model(Flip(axis=-1), input_data=input)

[GitHub] [tvm] masahi commented on a change in pull request #8443: Support aten::grid_sampler, aten::im2col, Fix bug aten::Float

2021-07-13 Thread GitBox
masahi commented on a change in pull request #8443: URL: https://github.com/apache/tvm/pull/8443#discussion_r669301896 ## File path: tests/python/frontend/pytorch/test_forward.py ## @@ -3912,6 +3912,109 @@ def forward(self, x): verify_model(Flip(axis=-1), input_data=input)

[GitHub] [tvm] masahi opened a new pull request #8466: [CUDA] Initial support for dynamic shared memory

2021-07-13 Thread GitBox
masahi opened a new pull request #8466: URL: https://github.com/apache/tvm/pull/8466 Only one allocation is allowed for now. -- 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.

[GitHub] [tvm] comaniac commented on pull request #8457: [CUDA] Improve injective schedule to enable half2

2021-07-13 Thread GitBox
comaniac commented on pull request #8457: URL: https://github.com/apache/tvm/pull/8457#issuecomment-879500795 Thanks @vinx13 @jcf94 -- 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

[tvm] branch main updated: [CUDA] Improve injective schedule to enable half2 (#8457)

2021-07-13 Thread comaniac
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 5c1a1cf [CUDA] Improve injective schedule to enable

[GitHub] [tvm] comaniac merged pull request #8457: [CUDA] Improve injective schedule to enable half2

2021-07-13 Thread GitBox
comaniac merged pull request #8457: URL: https://github.com/apache/tvm/pull/8457 -- 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.

[GitHub] [tvm] comaniac commented on a change in pull request #8464: [BYOC] add multi functions support in partition pass

2021-07-13 Thread GitBox
comaniac commented on a change in pull request #8464: URL: https://github.com/apache/tvm/pull/8464#discussion_r669175601 ## File path: src/relay/transforms/partition_graph.cc ## @@ -116,9 +116,11 @@ class Partitioner : public MixedModeMutator { for (auto f : module->functi

[GitHub] [tvm] zxy844288792 commented on a change in pull request #8464: [BYOC] add multi functions support in partition pass

2021-07-13 Thread GitBox
zxy844288792 commented on a change in pull request #8464: URL: https://github.com/apache/tvm/pull/8464#discussion_r669168791 ## File path: src/relay/transforms/partition_graph.cc ## @@ -301,7 +302,7 @@ class Partitioner : public MixedModeMutator { } std::string targ

[GitHub] [tvm] zxy844288792 commented on a change in pull request #8464: [BYOC] add multi functions support in partition pass

2021-07-13 Thread GitBox
zxy844288792 commented on a change in pull request #8464: URL: https://github.com/apache/tvm/pull/8464#discussion_r669168686 ## File path: src/relay/analysis/annotated_region_set.cc ## @@ -76,17 +76,21 @@ void AnnotatedRegionSetNode::AddToRegion(AnnotatedRegion dest, const Exp

[GitHub] [tvm] YuchenJin opened a new pull request #8465: [Fix] Remove unused variable in GraphExecutorCodegen

2021-07-13 Thread GitBox
YuchenJin opened a new pull request #8465: URL: https://github.com/apache/tvm/pull/8465 `lowered_funcs_` is unused after the compile engine refactoring. -- 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

[GitHub] [tvm] AndrewZhaoLuo commented on a change in pull request #8456: [Relay][Frontend][ONNX] Add ConvInteger support.

2021-07-13 Thread GitBox
AndrewZhaoLuo commented on a change in pull request #8456: URL: https://github.com/apache/tvm/pull/8456#discussion_r669158458 ## File path: python/tvm/relay/frontend/onnx.py ## @@ -3193,6 +3196,79 @@ def get_scalar(x, dtype="float32"): return _qnn.op.quantize(out, c_sc

[GitHub] [tvm] AndrewZhaoLuo commented on a change in pull request #8456: [Relay][Frontend][ONNX] Add ConvInteger support.

2021-07-13 Thread GitBox
AndrewZhaoLuo commented on a change in pull request #8456: URL: https://github.com/apache/tvm/pull/8456#discussion_r669158458 ## File path: python/tvm/relay/frontend/onnx.py ## @@ -3193,6 +3196,79 @@ def get_scalar(x, dtype="float32"): return _qnn.op.quantize(out, c_sc

[GitHub] [tvm] comaniac commented on a change in pull request #8464: [BYOC] add multi functions support in partition pass

2021-07-13 Thread GitBox
comaniac commented on a change in pull request #8464: URL: https://github.com/apache/tvm/pull/8464#discussion_r669156343 ## File path: src/relay/analysis/annotated_region_set.cc ## @@ -76,17 +76,21 @@ void AnnotatedRegionSetNode::AddToRegion(AnnotatedRegion dest, const Expr& e

[GitHub] [tvm] zxy844288792 commented on a change in pull request #8464: [BYOC] add multi functions support in partition pass

2021-07-13 Thread GitBox
zxy844288792 commented on a change in pull request #8464: URL: https://github.com/apache/tvm/pull/8464#discussion_r669155020 ## File path: src/relay/analysis/annotated_region_set.cc ## @@ -213,12 +217,19 @@ class AnnotatedRegionSet::Creator : protected MixedModeVisitor { co

[GitHub] [tvm] zxy844288792 commented on a change in pull request #8464: [BYOC] add multi functions support in partition pass

2021-07-13 Thread GitBox
zxy844288792 commented on a change in pull request #8464: URL: https://github.com/apache/tvm/pull/8464#discussion_r669154906 ## File path: src/relay/transforms/partition_graph.cc ## @@ -301,7 +302,7 @@ class Partitioner : public MixedModeMutator { } std::string targ

[GitHub] [tvm] schilkunda-amba commented on pull request #8455: [Relay] Support resize in the ONNX conversion

2021-07-13 Thread GitBox
schilkunda-amba commented on pull request #8455: URL: https://github.com/apache/tvm/pull/8455#issuecomment-879449950 > Interesting. We run this onnx file in CI and make sure it imports correctly: https://github.com/onnx/onnx/blob/v1.6.0/onnx/backend/test/data/node/test_resize_upsample_size

[GitHub] [tvm] zxy844288792 commented on a change in pull request #8464: [BYOC] add multi functions support in partition pass

2021-07-13 Thread GitBox
zxy844288792 commented on a change in pull request #8464: URL: https://github.com/apache/tvm/pull/8464#discussion_r669153705 ## File path: src/relay/analysis/annotated_region_set.cc ## @@ -76,17 +76,21 @@ void AnnotatedRegionSetNode::AddToRegion(AnnotatedRegion dest, const Exp

[GitHub] [tvm] mbrookhart commented on pull request #8455: [Relay] Support resize in the ONNX conversion

2021-07-13 Thread GitBox
mbrookhart commented on pull request #8455: URL: https://github.com/apache/tvm/pull/8455#issuecomment-879445759 Interesting. We run this onnx file in CI and make sure it imports correctly: https://github.com/onnx/onnx/blob/v1.6.0/onnx/backend/test/data/node/test_resize_upsample_sizes_neares

[GitHub] [tvm] comaniac commented on a change in pull request #8464: [BYOC] add multi functions support in partition pass

2021-07-13 Thread GitBox
comaniac commented on a change in pull request #8464: URL: https://github.com/apache/tvm/pull/8464#discussion_r669130328 ## File path: src/relay/analysis/annotated_region_set.cc ## @@ -213,12 +217,19 @@ class AnnotatedRegionSet::Creator : protected MixedModeVisitor { const

[tvm] branch main updated (1a26733 -> 73b38e8)

2021-07-13 Thread masahi
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 1a26733 [Refactor] Enforce attaching storage scope to PointerType (#8366) add 73b38e8 [Fix] Explicitly retain `__h

[GitHub] [tvm] masahi merged pull request #8449: [Fix] Explicitly retain `__hash__` of `StringImm`

2021-07-13 Thread GitBox
masahi merged pull request #8449: URL: https://github.com/apache/tvm/pull/8449 -- 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...

[GitHub] [tvm] zxy844288792 opened a new pull request #8464: [BYOC] add multi functions support in partition pass

2021-07-13 Thread GitBox
zxy844288792 opened a new pull request #8464: URL: https://github.com/apache/tvm/pull/8464 If a module contains multiple functions, the partition pass fail with this kind of message: ```Check failed: (!module_->ContainGlobalVar(fname)) is false: Global function ccompiler_0 already exists``

[GitHub] [tvm] schilkunda-amba commented on pull request #8455: [Relay] Support resize in the ONNX conversion

2021-07-13 Thread GitBox
schilkunda-amba commented on pull request #8455: URL: https://github.com/apache/tvm/pull/8455#issuecomment-879414215 > This looks fine to me. I am curious what the use case for this pass is :) @mbrookhart Here is the PR for this: https://discuss.tvm.apache.org/t/rfc-relay-to-onnx/610

[tvm] branch ci-docker-staging updated (7f5a959 -> 38b3165)

2021-07-13 Thread mbrookhart
This is an automated email from the ASF dual-hosted git repository. mbrookhart pushed a change to branch ci-docker-staging in repository https://gitbox.apache.org/repos/asf/tvm.git. from 7f5a959 next try at docker images add 38b3165 manage TF memory use in TF1 tests No new revisions

[GitHub] [tvm] zxy844288792 commented on pull request #8454: [Frontend, Tensorflow2] Added support for TensorList ops

2021-07-13 Thread GitBox
zxy844288792 commented on pull request #8454: URL: https://github.com/apache/tvm/pull/8454#issuecomment-879400436 @kevinthesun @yongwww @zhiics @trevor-m @masahi -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the UR

[GitHub] [tvm] masahi opened a new pull request #8463: [Refactor] Remove scope attribute from Buffer class

2021-07-13 Thread GitBox
masahi opened a new pull request #8463: URL: https://github.com/apache/tvm/pull/8463 -- 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-unsub

[GitHub] [tvm] masahi commented on pull request #8447: Vc/pytorch lstm

2021-07-13 Thread GitBox
masahi commented on pull request #8447: URL: https://github.com/apache/tvm/pull/8447#issuecomment-879375131 @vvchernov Both failures look new to me, they can be flaky tests. Can you try running CI again (rebase and push)? -- This is an automated message from the Apache Git Service. To re

[GitHub] [tvm] vinx13 commented on pull request #8366: [Refactor] Enforce attaching storage scope to PointerType

2021-07-13 Thread GitBox
vinx13 commented on pull request #8366: URL: https://github.com/apache/tvm/pull/8366#issuecomment-879346109 Thanks @masahi @kparzysz-quic @csullivan @tqchen this is merged -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[tvm] branch main updated (f62917e -> 1a26733)

2021-07-13 Thread wuwei
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 f62917e [TOPI] Add support for arbitrary dtypes to CSRMV and CSRMM (#8437) add 1a26733 [Refactor] Enforce attaching

[GitHub] [tvm] vinx13 merged pull request #8366: [Refactor] Enforce attaching storage scope to PointerType

2021-07-13 Thread GitBox
vinx13 merged pull request #8366: URL: https://github.com/apache/tvm/pull/8366 -- 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] branch main updated (136f218 -> f62917e)

2021-07-13 Thread areusch
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 136f218 [Relay][ONNX] Batch_matmul to dense optimization (#8440) add f62917e [TOPI] Add support for arbitrary dtyp

[GitHub] [tvm] areusch merged pull request #8437: [TOPI] Add support for arbitrary dtypes to CSRMV and CSRMM

2021-07-13 Thread GitBox
areusch merged pull request #8437: URL: https://github.com/apache/tvm/pull/8437 -- 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..

[GitHub] [tvm] areusch closed issue #8406: [BUG] topi.sparse.csrmv only accepts float32, but not other data types

2021-07-13 Thread GitBox
areusch closed issue #8406: URL: https://github.com/apache/tvm/issues/8406 -- 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

[GitHub] [tvm] vvchernov commented on pull request #8447: Vc/pytorch lstm

2021-07-13 Thread GitBox
vvchernov commented on pull request #8447: URL: https://github.com/apache/tvm/pull/8447#issuecomment-879329631 @masahi @jwfromm Hello guys, I have fixed all issues related to lint and resolved merge conflicts, but tvm-ci failed. After that I chose main with success status in jenkins (this

[GitHub] [tvm] Lunderberg opened a new pull request #8462: [UnitTests] Minor fixes to unit tests for cudnn/vulkan targets

2021-07-13 Thread GitBox
Lunderberg opened a new pull request #8462: URL: https://github.com/apache/tvm/pull/8462 - Marked tests as @requires_cudnn to avoid failure on platforms without cudnn. - Replaced target "vulkan" with "vulkan -from_device=0" -- This is an automated message from the Apache Git Servi

[GitHub] [tvm] schilkunda-amba commented on a change in pull request #8455: [Relay] Support resize in the ONNX conversion

2021-07-13 Thread GitBox
schilkunda-amba commented on a change in pull request #8455: URL: https://github.com/apache/tvm/pull/8455#discussion_r669026935 ## File path: tests/python/contrib/test_onnx.py ## @@ -655,6 +655,26 @@ def verify_cast(dshape, dtype): verify_cast(i, o_dtype) +def

[GitHub] [tvm] schilkunda-amba commented on a change in pull request #8455: [Relay] Support resize in the ONNX conversion

2021-07-13 Thread GitBox
schilkunda-amba commented on a change in pull request #8455: URL: https://github.com/apache/tvm/pull/8455#discussion_r669026790 ## File path: python/tvm/contrib/target/onnx.py ## @@ -662,6 +662,86 @@ def convert_attributes(cls, attrs): return {"to": getattr(TensorProto

[GitHub] [tvm] trevor-m opened a new pull request #8461: [BYOC][TensorRT] Fix dynamic batching when use_implicit_batch=False

2021-07-13 Thread GitBox
trevor-m opened a new pull request #8461: URL: https://github.com/apache/tvm/pull/8461 Fixes error for subgraphs with dynamic batch size when use_implicit_batch=False. Added test case. Also fixes deserialization of cached engine when batch size is not 1. -- This is an automated me

[GitHub] [tvm] comaniac commented on pull request #8455: [Relay] Support resize in the ONNX conversion

2021-07-13 Thread GitBox
comaniac commented on pull request #8455: URL: https://github.com/apache/tvm/pull/8455#issuecomment-879315407 @mbrookhart could you help take a look if you got a chance? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [tvm] schilkunda-amba commented on pull request #8455: [Relay] Support resize in the ONNX conversion

2021-07-13 Thread GitBox
schilkunda-amba commented on pull request #8455: URL: https://github.com/apache/tvm/pull/8455#issuecomment-879314253 @comaniac @trevor-m I have made the changes (catch error using else and updating the test for coord trans and rounding method). I see that it the conversion fails (as in

[GitHub] [tvm] schilkunda-amba commented on pull request #8455: [Relay] Support resize in the ONNX conversion

2021-07-13 Thread GitBox
schilkunda-amba commented on pull request #8455: URL: https://github.com/apache/tvm/pull/8455#issuecomment-879312115 > Can we also support resize in other dimensions? Sure, I will look into it. -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [tvm] areusch commented on issue #8177: Rebuild ci-arm, ci-cpu, and ci-gpu container

2021-07-13 Thread GitBox
areusch commented on issue #8177: URL: https://github.com/apache/tvm/issues/8177#issuecomment-879295272 It may have that, I think there are 2 executors per GPU node. On Tue, Jul 13, 2021 at 10:52 AM Leandro Nunes ***@***.***> wrote: > I don't see any excessive memory use whe

[GitHub] [tvm] comaniac commented on pull request #8440: [Relay][ONNX] Batch_matmul to dense optimization

2021-07-13 Thread GitBox
comaniac commented on pull request #8440: URL: https://github.com/apache/tvm/pull/8440#issuecomment-879294697 Thanks @ymwangg -- 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 commen

[tvm] branch main updated (d67514b -> 136f218)

2021-07-13 Thread comaniac
This is an automated email from the ASF dual-hosted git repository. comaniac pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/tvm.git. from d67514b [PROFILING] Use PAPI to collect hardware performance counters on CPU and CUDA (#7983) add 136f218 [Relay

[GitHub] [tvm] comaniac merged pull request #8440: [Relay][ONNX] Batch_matmul to dense optimization

2021-07-13 Thread GitBox
comaniac merged pull request #8440: URL: https://github.com/apache/tvm/pull/8440 -- 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.

[GitHub] [tvm] jwfromm commented on a change in pull request #8456: [Relay][Frontend][ONNX] Add ConvInteger support.

2021-07-13 Thread GitBox
jwfromm commented on a change in pull request #8456: URL: https://github.com/apache/tvm/pull/8456#discussion_r668992231 ## File path: python/tvm/relay/frontend/onnx.py ## @@ -3193,6 +3196,79 @@ def get_scalar(x, dtype="float32"): return _qnn.op.quantize(out, c_scale, c

[GitHub] [tvm] AndrewZhaoLuo opened a new pull request #8460: [AMP] Add default op attribute registration to __init__.py

2021-07-13 Thread GitBox
AndrewZhaoLuo opened a new pull request #8460: URL: https://github.com/apache/tvm/pull/8460 This will make it so when we do `from tvm.relay.transform import ToMixedPrecision` we will also by default import the default op attributes for the pass. This will hopefully avoid people suffering f

[GitHub] [tvm] leandron edited a comment on issue #8177: Rebuild ci-arm, ci-cpu, and ci-gpu container

2021-07-13 Thread GitBox
leandron edited a comment on issue #8177: URL: https://github.com/apache/tvm/issues/8177#issuecomment-879284196 > I don't see any excessive memory use when I put the test on a loop locally or run the full tensorflow test suite locally with this version of TF... Does Jenkins has more

[GitHub] [tvm] leandron commented on issue #8177: Rebuild ci-arm, ci-cpu, and ci-gpu container

2021-07-13 Thread GitBox
leandron commented on issue #8177: URL: https://github.com/apache/tvm/issues/8177#issuecomment-879284196 > I don't see any excessive memory use when I put the test on a loop locally or run the full tensorflow test suite locally with this version of TF... Does Jenkins has more than on

[GitHub] [tvm] AndrewZhaoLuo commented on a change in pull request #8456: [Relay][Frontend][ONNX] Add ConvInteger support.

2021-07-13 Thread GitBox
AndrewZhaoLuo commented on a change in pull request #8456: URL: https://github.com/apache/tvm/pull/8456#discussion_r668979372 ## File path: python/tvm/relay/frontend/onnx.py ## @@ -3193,6 +3196,79 @@ def get_scalar(x, dtype="float32"): return _qnn.op.quantize(out, c_sc

[GitHub] [tvm] areusch commented on a change in pull request #8451: [UnitTests] Require cached fixtures to be copy-able, with opt-in.

2021-07-13 Thread GitBox
areusch commented on a change in pull request #8451: URL: https://github.com/apache/tvm/pull/8451#discussion_r668949357 ## File path: python/tvm/testing.py ## @@ -1160,6 +1162,43 @@ def wraps(func): return wraps(func) +class _DeepCopyAllowedClasses(dict): +def __in

[GitHub] [tvm] AndrewZhaoLuo commented on issue #8296: [AMP] Tracking Issue for Mixed Precision Pass

2021-07-13 Thread GitBox
AndrewZhaoLuo commented on issue #8296: URL: https://github.com/apache/tvm/issues/8296#issuecomment-879273695 cc @Lunderberg -- 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

[tvm] branch main updated (957cc12 -> d67514b)

2021-07-13 Thread areusch
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 957cc12 [Relay] Modify create_executor to pass params (#8418) add d67514b [PROFILING] Use PAPI to collect hardware

[GitHub] [tvm] areusch merged pull request #7983: [PROFILING] Use PAPI to collect hardware performance counters on CPU and CUDA

2021-07-13 Thread GitBox
areusch merged pull request #7983: URL: https://github.com/apache/tvm/pull/7983 -- 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..

[GitHub] [tvm] jwfromm commented on pull request #8456: [Relay][Frontend][ONNX] Add ConvInteger support and fix some ConvTranspose padding bugs.

2021-07-13 Thread GitBox
jwfromm commented on pull request #8456: URL: https://github.com/apache/tvm/pull/8456#issuecomment-879240885 @mbrookhart that makes sense, I've removed all changes to convtranspose from this PR. -- This is an automated message from the Apache Git Service. To respond to the message, pleas

[GitHub] [tvm] comaniac commented on pull request #8457: [CUDA] Improve injective schedule to enable half2

2021-07-13 Thread GitBox
comaniac commented on pull request #8457: URL: https://github.com/apache/tvm/pull/8457#issuecomment-879236252 > Thanks! > I'm thinking that if loop partition can solve this problem? @comaniac @junrushao1994 > For example, with a dimension of 301, partition it to a 300 loop and a sing

[GitHub] [tvm] mbrookhart commented on pull request #8456: [Relay][Frontend][ONNX] Add ConvInteger support and fix some ConvTranspose padding bugs.

2021-07-13 Thread GitBox
mbrookhart commented on pull request #8456: URL: https://github.com/apache/tvm/pull/8456#issuecomment-879214900 I'm having trouble getting the docker images in CI updated due to a TF update. -- This is an automated message from the Apache Git Service. To respond to the message, please lo

[GitHub] [tvm] mbrookhart commented on pull request #8456: [Relay][Frontend][ONNX] Add ConvInteger support and fix some ConvTranspose padding bugs.

2021-07-13 Thread GitBox
mbrookhart commented on pull request #8456: URL: https://github.com/apache/tvm/pull/8456#issuecomment-879214643 @jwfromm I have some of the same changes for conv_transpose in #8304 . Unfortunately, the version of onnxrutnime in CI doesn't support this properly, so we need to wait until we

[tvm] branch main updated (80f48c7 -> 957cc12)

2021-07-13 Thread ziheng
This is an automated email from the ASF dual-hosted git repository. ziheng pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/tvm.git. from 80f48c7 [microTVM] Fix Stack Size Issue for Zephyr AOT Demo on Physical Hardware (#8453) add 957cc12 [Relay] Modif

[GitHub] [tvm] ZihengJiang merged pull request #8418: [Relay] Modify create_executor to pass params

2021-07-13 Thread GitBox
ZihengJiang merged pull request #8418: URL: https://github.com/apache/tvm/pull/8418 -- 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-unsubs

[GitHub] [tvm] ZihengJiang commented on pull request #8418: [Relay] Modify create_executor to pass params

2021-07-13 Thread GitBox
ZihengJiang commented on pull request #8418: URL: https://github.com/apache/tvm/pull/8418#issuecomment-879210310 Merged now. Thanks @mikepapadim -- 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

[GitHub] [tvm] Lunderberg opened a new pull request #8459: [Docs] Corrected typo in googletest build instructions.

2021-07-13 Thread GitBox
Lunderberg opened a new pull request #8459: URL: https://github.com/apache/tvm/pull/8459 Incorrectly specified "MAKE_SHARED_LIBS" option instead of "BUILD_SHARED_LIBS". In future, perhaps googletest should be pulled into 3rdparty submodules to minimize the setup of a dev environment. --

[GitHub] [tvm] leandron commented on pull request #8453: [microTVM] Fix Stack Size Issue for Zephyr AOT Demo on Physical Hardware

2021-07-13 Thread GitBox
leandron commented on pull request #8453: URL: https://github.com/apache/tvm/pull/8453#issuecomment-878948703 This is now merged, thanks @mehrdadh @Mousius @areusch! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[tvm] branch main updated (807373c -> 80f48c7)

2021-07-13 Thread leandron
This is an automated email from the ASF dual-hosted git repository. leandron pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/tvm.git. from 807373c Add qnn batch_matmul operator (#8401) add 80f48c7 [microTVM] Fix Stack Size Issue for Zephyr AOT Demo on

[GitHub] [tvm] leandron merged pull request #8453: [microTVM] Fix Stack Size Issue for Zephyr AOT Demo on Physical Hardware

2021-07-13 Thread GitBox
leandron merged pull request #8453: URL: https://github.com/apache/tvm/pull/8453 -- 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.

[GitHub] [tvm] delldu commented on a change in pull request #8443: Support aten::grid_sampler, aten::im2col, Fix bug aten::Float

2021-07-13 Thread GitBox
delldu commented on a change in pull request #8443: URL: https://github.com/apache/tvm/pull/8443#discussion_r668549131 ## File path: tests/python/frontend/pytorch/test_forward.py ## @@ -3912,6 +3912,109 @@ def forward(self, x): verify_model(Flip(axis=-1), input_data=input)

[GitHub] [tvm] wyc-ruiker commented on a change in pull request #8443: Support aten::grid_sampler, aten::im2col, Fix bug aten::Float

2021-07-13 Thread GitBox
wyc-ruiker commented on a change in pull request #8443: URL: https://github.com/apache/tvm/pull/8443#discussion_r668572817 ## File path: tests/python/frontend/pytorch/test_forward.py ## @@ -3912,6 +3912,109 @@ def forward(self, x): verify_model(Flip(axis=-1), input_data=in

[GitHub] [tvm] delldu commented on a change in pull request #8443: Support aten::grid_sampler, aten::im2col, Fix bug aten::Float

2021-07-13 Thread GitBox
delldu commented on a change in pull request #8443: URL: https://github.com/apache/tvm/pull/8443#discussion_r668549131 ## File path: tests/python/frontend/pytorch/test_forward.py ## @@ -3912,6 +3912,109 @@ def forward(self, x): verify_model(Flip(axis=-1), input_data=input)

[GitHub] [tvm] mikepapadim commented on a change in pull request #8418: [Relay] Modify create_executor to pass params

2021-07-13 Thread GitBox
mikepapadim commented on a change in pull request #8418: URL: https://github.com/apache/tvm/pull/8418#discussion_r668544863 ## File path: python/tvm/relay/build_module.py ## @@ -555,6 +559,9 @@ def create_executor(kind="debug", mod=None, device=None, target="llvm"): else:

[GitHub] [tvm-rfcs] junrushao1994 commented on pull request #5: [RFC] Meta Schedule (AutoTIR)

2021-07-13 Thread GitBox
junrushao1994 commented on pull request #5: URL: https://github.com/apache/tvm-rfcs/pull/5#issuecomment-878885449 @areusch Thank you sooo much for reviewing this RFC! I really appreciate it that you went over the text very carefully and provided very helpful suggestions! Just finished a pa

[GitHub] [tvm-rfcs] junrushao1994 commented on a change in pull request #5: [RFC] Meta Schedule (AutoTIR)

2021-07-13 Thread GitBox
junrushao1994 commented on a change in pull request #5: URL: https://github.com/apache/tvm-rfcs/pull/5#discussion_r668540347 ## File path: rfcs/0001-meta-schedule-autotensorir.md ## @@ -0,0 +1,444 @@ + + + + + + + + + + + + + + + + + +* Feature Name: Meta Schedule (Formerly Aut

[GitHub] [tvm-rfcs] junrushao1994 commented on a change in pull request #5: [RFC] Meta Schedule (AutoTIR)

2021-07-13 Thread GitBox
junrushao1994 commented on a change in pull request #5: URL: https://github.com/apache/tvm-rfcs/pull/5#discussion_r668525299 ## File path: rfcs/0001-meta-schedule-autotensorir.md ## @@ -0,0 +1,444 @@ + + + + + + + + + + + + + + + + + +* Feature Name: Meta Schedule (Formerly Aut

[GitHub] [tvm-rfcs] junrushao1994 commented on a change in pull request #5: [RFC] Meta Schedule (AutoTIR)

2021-07-13 Thread GitBox
junrushao1994 commented on a change in pull request #5: URL: https://github.com/apache/tvm-rfcs/pull/5#discussion_r668524060 ## File path: rfcs/0001-meta-schedule-autotensorir.md ## @@ -0,0 +1,339 @@ + + + + + + + + + + + + + + + + + +* Feature Name: Meta Schedule (AutoTIR) +*

[GitHub] [tvm-rfcs] junrushao1994 commented on a change in pull request #5: [RFC] Meta Schedule (AutoTIR)

2021-07-13 Thread GitBox
junrushao1994 commented on a change in pull request #5: URL: https://github.com/apache/tvm-rfcs/pull/5#discussion_r668518322 ## File path: rfcs/0001-meta-schedule-autotensorir.md ## @@ -0,0 +1,339 @@ + + + + + + + + + + + + + + + + + +* Feature Name: Meta Schedule (AutoTIR) +*

[GitHub] [tvm-rfcs] junrushao1994 commented on a change in pull request #5: [RFC] Meta Schedule (AutoTIR)

2021-07-13 Thread GitBox
junrushao1994 commented on a change in pull request #5: URL: https://github.com/apache/tvm-rfcs/pull/5#discussion_r668514392 ## File path: rfcs/0001-meta-schedule-autotensorir.md ## @@ -0,0 +1,444 @@ + + + + + + + + + + + + + + + + + +* Feature Name: Meta Schedule (Formerly Aut

[GitHub] [tvm-rfcs] junrushao1994 commented on a change in pull request #5: [RFC] Meta Schedule (AutoTIR)

2021-07-13 Thread GitBox
junrushao1994 commented on a change in pull request #5: URL: https://github.com/apache/tvm-rfcs/pull/5#discussion_r668508988 ## File path: rfcs/0001-meta-schedule-autotensorir.md ## @@ -0,0 +1,444 @@ + + + + + + + + + + + + + + + + + +* Feature Name: Meta Schedule (Formerly Aut

[GitHub] [tvm-rfcs] junrushao1994 commented on a change in pull request #5: [RFC] Meta Schedule (AutoTIR)

2021-07-13 Thread GitBox
junrushao1994 commented on a change in pull request #5: URL: https://github.com/apache/tvm-rfcs/pull/5#discussion_r668506959 ## File path: rfcs/0001-meta-schedule-autotensorir.md ## @@ -0,0 +1,444 @@ + + + + + + + + + + + + + + + + + +* Feature Name: Meta Schedule (Formerly Aut

[GitHub] [tvm-rfcs] junrushao1994 commented on a change in pull request #5: [RFC] Meta Schedule (AutoTIR)

2021-07-13 Thread GitBox
junrushao1994 commented on a change in pull request #5: URL: https://github.com/apache/tvm-rfcs/pull/5#discussion_r668383535 ## File path: rfcs/0001-meta-schedule-autotensorir.md ## @@ -0,0 +1,444 @@ + + + + + + + + + + + + + + + + + +* Feature Name: Meta Schedule (Formerly Aut

[GitHub] [tvm-rfcs] junrushao1994 commented on a change in pull request #5: [RFC] Meta Schedule (AutoTIR)

2021-07-13 Thread GitBox
junrushao1994 commented on a change in pull request #5: URL: https://github.com/apache/tvm-rfcs/pull/5#discussion_r668502241 ## File path: rfcs/0001-meta-schedule-autotensorir.md ## @@ -0,0 +1,444 @@ + + + + + + + + + + + + + + + + + +* Feature Name: Meta Schedule (Formerly Aut

[GitHub] [tvm-rfcs] junrushao1994 commented on a change in pull request #5: [RFC] Meta Schedule (AutoTIR)

2021-07-13 Thread GitBox
junrushao1994 commented on a change in pull request #5: URL: https://github.com/apache/tvm-rfcs/pull/5#discussion_r668500696 ## File path: rfcs/0001-meta-schedule-autotensorir.md ## @@ -0,0 +1,297 @@ +* Feature Name: Meta Schedule (AutoTensorIR) +* Start Date: 2021-05-28 +* RFC

[GitHub] [tvm-rfcs] junrushao1994 commented on a change in pull request #5: [RFC] Meta Schedule (AutoTIR)

2021-07-13 Thread GitBox
junrushao1994 commented on a change in pull request #5: URL: https://github.com/apache/tvm-rfcs/pull/5#discussion_r668494320 ## File path: rfcs/0001-meta-schedule-autotensorir.md ## @@ -0,0 +1,444 @@ + + + + + + + + + + + + + + + + + +* Feature Name: Meta Schedule (Formerly Aut

[GitHub] [tvm-rfcs] junrushao1994 commented on a change in pull request #5: [RFC] Meta Schedule (AutoTIR)

2021-07-13 Thread GitBox
junrushao1994 commented on a change in pull request #5: URL: https://github.com/apache/tvm-rfcs/pull/5#discussion_r668494320 ## File path: rfcs/0001-meta-schedule-autotensorir.md ## @@ -0,0 +1,444 @@ + + + + + + + + + + + + + + + + + +* Feature Name: Meta Schedule (Formerly Aut

[GitHub] [tvm-rfcs] junrushao1994 commented on a change in pull request #5: [RFC] Meta Schedule (AutoTIR)

2021-07-13 Thread GitBox
junrushao1994 commented on a change in pull request #5: URL: https://github.com/apache/tvm-rfcs/pull/5#discussion_r668494320 ## File path: rfcs/0001-meta-schedule-autotensorir.md ## @@ -0,0 +1,444 @@ + + + + + + + + + + + + + + + + + +* Feature Name: Meta Schedule (Formerly Aut

[GitHub] [tvm] Hzfengsy commented on pull request #8458: [TVMSCRIPT] Add tir.ceil node in tvm script

2021-07-13 Thread GitBox
Hzfengsy commented on pull request #8458: URL: https://github.com/apache/tvm/pull/8458#issuecomment-878841268 Please check the printer and add a simple testcase in `test_tvmscript_round_trip.py` -- This is an automated message from the Apache Git Service. To respond to the message, pleas

[GitHub] [tvm] UniverseFly commented on a change in pull request #8449: [Fix] Explicitly retain `__hash__` of `StringImm`

2021-07-13 Thread GitBox
UniverseFly commented on a change in pull request #8449: URL: https://github.com/apache/tvm/pull/8449#discussion_r668491558 ## File path: python/tvm/tir/expr.py ## @@ -577,6 +577,8 @@ def __ne__(self, other): return self.value != other.value return self.va

[GitHub] [tvm-rfcs] junrushao1994 commented on a change in pull request #5: [RFC] Meta Schedule (AutoTIR)

2021-07-13 Thread GitBox
junrushao1994 commented on a change in pull request #5: URL: https://github.com/apache/tvm-rfcs/pull/5#discussion_r668491013 ## File path: rfcs/0001-meta-schedule-autotensorir.md ## @@ -0,0 +1,444 @@ + + + + + + + + + + + + + + + + + +* Feature Name: Meta Schedule (Formerly Aut

[GitHub] [tvm] Beya2019 opened a new pull request #8458: [TVMSCRIPT] Add tir.ceil node in tvm script

2021-07-13 Thread GitBox
Beya2019 opened a new pull request #8458: URL: https://github.com/apache/tvm/pull/8458 Tvm script lacks tir.min node, we can use it like below with tir.block([128, 128], "") as [vi, vj]: A[vi, vj] = tir.ceil(2.5) The result: with tir.block([128, 12

[GitHub] [tvm-rfcs] junrushao1994 commented on a change in pull request #5: [RFC] Meta Schedule (AutoTIR)

2021-07-13 Thread GitBox
junrushao1994 commented on a change in pull request #5: URL: https://github.com/apache/tvm-rfcs/pull/5#discussion_r668482353 ## File path: rfcs/0001-meta-schedule-autotensorir.md ## @@ -0,0 +1,444 @@ + + + + + + + + + + + + + + + + + +* Feature Name: Meta Schedule (Formerly Aut