[GitHub] [tvm] chunit-quic opened a new pull request #9723: [Frontend] Add Span filling for frontends to Relay

2021-12-13 Thread GitBox
chunit-quic opened a new pull request #9723: URL: https://github.com/apache/tvm/pull/9723 * Add a common span filling feature for tf1/2, tflite and pytorch. * Add test case for Span filling in each frontend. * Expose Tuple and TupleGetItem to python end Hi community,

[GitHub] [tvm] grant-arm opened a new pull request #9725: [microNPU] Update Arm(R) Ethos(TM)-U55 NPU demo README

2021-12-13 Thread GitBox
grant-arm opened a new pull request #9725: URL: https://github.com/apache/tvm/pull/9725 An [issue was raised in the Discuss forums](https://discuss.tvm.apache.org/t/cannot-run-arm-ethosu-demo/11684/) where it isn't clear what options should be set when building TVM from source for

[GitHub] [tvm] hgt312 commented on issue #9237: [Bug] [PYTORCH] [FRONTEND] Torch and tvm output different

2021-12-13 Thread GitBox
hgt312 commented on issue #9237: URL: https://github.com/apache/tvm/issues/9237#issuecomment-992428813 The issue is about boolean indexing, see https://numpy.org/doc/stable/reference/arrays.indexing.html#boolean-array-indexing, while relay's op `adv_index` is integer array indexing.

[GitHub] [tvm] leandron merged pull request #9712: [TVMC] run: Don't use static path to find model.tar

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

[GitHub] [tvm] leandron commented on pull request #9722: [TVMC] Add --opt-level to compile mode

2021-12-13 Thread GitBox
leandron commented on pull request #9722: URL: https://github.com/apache/tvm/pull/9722#issuecomment-992305471 There was an infrastructure CI issue with your patch, related to #9705. Can you retriever CI on this PR, so that we can move forward and merge it once CI is green? -- This is

[GitHub] [tvm] lucagrementieri edited a comment on issue #9237: [Bug] [PYTORCH] [FRONTEND] Torch and tvm output different

2021-12-13 Thread GitBox
lucagrementieri edited a comment on issue #9237: URL: https://github.com/apache/tvm/issues/9237#issuecomment-943090057 I am facing a similar problem running the script `from_torch.py` published with the tutorial https://tvm.apache.org/docs/how_to/compile_models/from_pytorch.html.

[GitHub] [tvm] manupa-arm merged pull request #9708: Follow up from CMSIS-NN Pooling failure

2021-12-13 Thread GitBox
manupa-arm merged pull request #9708: URL: https://github.com/apache/tvm/pull/9708 -- 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:

[GitHub] [tvm] hgt312 commented on pull request #9717: [Bugfix][Op] Fix shape inference of adv_index

2021-12-13 Thread GitBox
hgt312 commented on pull request #9717: URL: https://github.com/apache/tvm/pull/9717#issuecomment-992404830 > Can you also check if this fixes the issue in #9237? it not fix that, that issue is about boolean index, see https://github.com/apache/tvm/pull/7373 I agree that we can

[GitHub] [tvm] ashutosh-arm edited a comment on pull request #9708: Follow up from CMSIS-NN Pooling failure

2021-12-13 Thread GitBox
ashutosh-arm edited a comment on pull request #9708: URL: https://github.com/apache/tvm/pull/9708#issuecomment-992279714 Could you folks @ekalda @manupa-arm please take a look at it? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [tvm] ashutosh-arm commented on pull request #9708: Follow up from CMSIS-NN Pooling failure

2021-12-13 Thread GitBox
ashutosh-arm commented on pull request #9708: URL: https://github.com/apache/tvm/pull/9708#issuecomment-992279714 Could you folks @ekalda @manupa-arm take a look at it? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [tvm] jiangjiajun opened a new pull request #9724: [Frontend][PaddlePaddle] Enhance paddlepaddle frontend with more operators

2021-12-13 Thread GitBox
jiangjiajun opened a new pull request #9724: URL: https://github.com/apache/tvm/pull/9724 Add 13 new operators - clip - elu - instance_norm - log1p - logsigmoid - logsoftmax - logsumexp - meshgrid - mv - pixel_shuffle - prelu - range - strided_slice

[GitHub] [tvm] manupa-arm commented on issue #9705: [CI Problem] CI node out of memory

2021-12-13 Thread GitBox
manupa-arm commented on issue #9705: URL: https://github.com/apache/tvm/issues/9705#issuecomment-992306803 Thanks! @leandron -- 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

[GitHub] [tvm] leandron merged pull request #9074: Improve tvmc error message from lazy-loading frontend imports

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

[tvm] branch main updated (c19f193 -> 5557b8c)

2021-12-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 c19f193 Follow up from CMSIS-NN pooling failure (#9708) add 5557b8c Improve tvmc error message from lazy-loading

[GitHub] [tvm] masahi commented on pull request #9717: [Bugfix][Op] Fix shape inference of adv_index

2021-12-13 Thread GitBox
masahi commented on pull request #9717: URL: https://github.com/apache/tvm/pull/9717#issuecomment-992411871 The CI has been dead since the last weekend. Please wait until it is back. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[tvm] branch main updated (2b35cfd -> 6efade6)

2021-12-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 2b35cfd Add Hexagon VTCM and discontiguous allocation support (#9525) add 6efade6 [TVMC] run: Don't use static

[GitHub] [tvm] manupa-arm edited a comment on issue #9713: [CI Image] Update PyTorch to v1.10 in GPU image

2021-12-13 Thread GitBox
manupa-arm edited a comment on issue #9713: URL: https://github.com/apache/tvm/issues/9713#issuecomment-992320714 Hi @masahi , FYI, I'd be careful with S3. I was trying to use the image from S2 in S3 as mentioned here : https://github.com/apache/tvm/issues/9659

[GitHub] [tvm] manupa-arm commented on pull request #9708: Follow up from CMSIS-NN Pooling failure

2021-12-13 Thread GitBox
manupa-arm commented on pull request #9708: URL: https://github.com/apache/tvm/pull/9708#issuecomment-992347969 Thanks! @ashutosh-arm -- 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

[tvm] branch main updated: Follow up from CMSIS-NN pooling failure (#9708)

2021-12-13 Thread manupa
This is an automated email from the ASF dual-hosted git repository. manupa 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 c19f193 Follow up from CMSIS-NN pooling failure

[GitHub] [tvm] lixiaoquan commented on pull request #9723: [Frontend] Add Span filling for frontends to Relay

2021-12-13 Thread GitBox
lixiaoquan commented on pull request #9723: URL: https://github.com/apache/tvm/pull/9723#issuecomment-992346440 I just feel that in `one to many` case, the original tensor/layer name should be attached to the last node in the group. Because that's where the computational result (and

[GitHub] [tvm] leandron commented on issue #9705: [CI Problem] CI node out of memory

2021-12-13 Thread GitBox
leandron commented on issue #9705: URL: https://github.com/apache/tvm/issues/9705#issuecomment-992303400 I temporarily disabled this machine on Jenkins, due to storage issues, FYI. https://ci.tlcpack.ai/computer/octo.aws.c4.44.242.35.246/ -- This is an automated message from the

[GitHub] [tvm] manupa-arm commented on issue #9659: [CI Image] Update ci_cpu to v0.80

2021-12-13 Thread GitBox
manupa-arm commented on issue #9659: URL: https://github.com/apache/tvm/issues/9659#issuecomment-992315290 This is blocked on https://github.com/apache/tvm/issues/9705 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [tvm] leeexyz commented on pull request #9722: [TVMC] Add --opt-level to compile mode

2021-12-13 Thread GitBox
leeexyz commented on pull request #9722: URL: https://github.com/apache/tvm/pull/9722#issuecomment-992314865 > There was an infrastructure CI issue with your patch, related to #9705. Can you retriever CI on this PR, so that we can move forward and merge it once CI is green? Thanks!

[GitHub] [tvm] manupa-arm commented on issue #9713: [CI Image] Update PyTorch to v1.10 in GPU image

2021-12-13 Thread GitBox
manupa-arm commented on issue #9713: URL: https://github.com/apache/tvm/issues/9713#issuecomment-992320714 Hi @masahi , FYI, I'd be careful with S3. I was trying to use the image from S2 in S3 as mentioned here : https://github.com/apache/tvm/issues/9659 Unfortunately, it

[GitHub] [tvm] manupa-arm edited a comment on issue #9713: [CI Image] Update PyTorch to v1.10 in GPU image

2021-12-13 Thread GitBox
manupa-arm edited a comment on issue #9713: URL: https://github.com/apache/tvm/issues/9713#issuecomment-992320714 Hi @masahi , FYI, I'd be careful with S3. I was trying to use the image from S2 in S3 as mentioned here : https://github.com/apache/tvm/issues/9659

[GitHub] [tvm] masahi commented on issue #9713: [CI Image] Update PyTorch to v1.10 in GPU image

2021-12-13 Thread GitBox
masahi commented on issue #9713: URL: https://github.com/apache/tvm/issues/9713#issuecomment-992330893 @manupa-arm Thanks, yes I'm aware of the recent CI outrage. Probably I'll work on this next week. -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [tvm] hgt312 commented on pull request #9717: [Bugfix][Op] Fix shape inference of adv_index

2021-12-13 Thread GitBox
hgt312 commented on pull request #9717: URL: https://github.com/apache/tvm/pull/9717#issuecomment-992406582 Another question, can the failed CI be retry? -- 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

[GitHub] [tvm] chunit-quic commented on pull request #9723: [Frontend] Add Span filling for frontends to Relay

2021-12-13 Thread GitBox
chunit-quic commented on pull request #9723: URL: https://github.com/apache/tvm/pull/9723#issuecomment-992451852 Hi @lixiaoquan, It's a good advice and easy to implement. :D If it tag the original name to the final expression is a better philosophy for users, I can change to

[GitHub] [tvm] Lunderberg commented on pull request #9727: [TE][TIR] Implement layout transformations, non-flat memory buffers

2021-12-13 Thread GitBox
Lunderberg commented on pull request #9727: URL: https://github.com/apache/tvm/pull/9727#issuecomment-992578126 @vinx13 @csullivan @tmoreau89 @adstraw @cconvey -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [tvm] MasterJH5574 opened a new pull request #9728: [BugFix] Fix a wrong use of `std::move()` in cross-thread reduction lowering

2021-12-13 Thread GitBox
MasterJH5574 opened a new pull request #9728: URL: https://github.com/apache/tvm/pull/9728 Find a wrong use of `std::move()`. Please see the changes for detail. The wrong use leads to undefined Var of a buffer's `data` field. Given C++ evaluates parameter expressions from the end to

[tvm] branch main updated (5557b8c -> cb34604)

2021-12-13 Thread mbrookhart
This is an automated email from the ASF dual-hosted git repository. mbrookhart pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/tvm.git. from 5557b8c Improve tvmc error message from lazy-loading frontend imports (#9074) add cb34604 [TOPI] Add generic

[GitHub] [tvm] manupa-arm commented on a change in pull request #9704: [USMP] Hill Climb allocator

2021-12-13 Thread GitBox
manupa-arm commented on a change in pull request #9704: URL: https://github.com/apache/tvm/pull/9704#discussion_r767889703 ## File path: src/tir/usmp/algo/hill_climb.cc ## @@ -0,0 +1,356 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more

[GitHub] [tvm-rfcs] Lunderberg commented on pull request #39: [RFC][TIR] Layout transformations on buffer access

2021-12-13 Thread GitBox
Lunderberg commented on pull request #39: URL: https://github.com/apache/tvm-rfcs/pull/39#issuecomment-992608499 Sounds good! I've updated with examples of scheduling with the returned new axes, which work in the implementation posted in

[GitHub] [tvm] manupa-arm commented on a change in pull request #9704: [USMP] Hill Climb allocator

2021-12-13 Thread GitBox
manupa-arm commented on a change in pull request #9704: URL: https://github.com/apache/tvm/pull/9704#discussion_r767851543 ## File path: src/tir/usmp/algo/hill_climb.cc ## @@ -0,0 +1,349 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more

[GitHub] [tvm] abdulazizm commented on issue #7971: Runtime error when tracing maskrcnn model: Only tensors, lists, tuples of tensors, or dictionary of tensors can be output from traced functions

2021-12-13 Thread GitBox
abdulazizm commented on issue #7971: URL: https://github.com/apache/tvm/issues/7971#issuecomment-992568025 > I will reformulate the question in a new issue, since this one is already closed... Did you raise the issue for this query? -- This is an automated message from the Apache

[GitHub] [tvm] Lunderberg opened a new pull request #9727: [TE][TIR] Implement layout transformations, non-flat memory buffers

2021-12-13 Thread GitBox
Lunderberg opened a new pull request #9727: URL: https://github.com/apache/tvm/pull/9727 This is an implementation of [RFC#0039](https://github.com/apache/tvm-rfcs/pull/39), which adds layout transformations to both flat and non-flat memory buffers. The commits in this PR are split

[GitHub] [tvm] chunit-quic edited a comment on pull request #9723: [Frontend] Add Span filling for frontends to Relay

2021-12-13 Thread GitBox
chunit-quic edited a comment on pull request #9723: URL: https://github.com/apache/tvm/pull/9723#issuecomment-992451852 Hi @lixiaoquan, It's a good advice and easy to implement. :D If tag the original name to the final expression is a better philosophy for users, I can change

[GitHub] [tvm] gromero opened a new pull request #9726: Fix typos in runtime comments

2021-12-13 Thread GitBox
gromero opened a new pull request #9726: URL: https://github.com/apache/tvm/pull/9726 Fix typos in comments about signal handling and entries in the runtime code. Signed-off-by: Gustavo Romero Thanks for contributing to TVM! Please refer to guideline

[GitHub] [tvm] abdulazizm commented on issue #8683: Error when using `partition_for_vitis_ai` with YOLOX ONNX model

2021-12-13 Thread GitBox
abdulazizm commented on issue #8683: URL: https://github.com/apache/tvm/issues/8683#issuecomment-992570402 > Hi @comaniac @jlamperez , > > I am following the

[GitHub] [tvm] daniperfer commented on issue #7971: Runtime error when tracing maskrcnn model: Only tensors, lists, tuples of tensors, or dictionary of tensors can be output from traced functions

2021-12-13 Thread GitBox
daniperfer commented on issue #7971: URL: https://github.com/apache/tvm/issues/7971#issuecomment-992612913 Yes, it's in #7990. -- 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

[GitHub] [tvm] mbrookhart merged pull request #9694: [TOPI] Add generic batch norm

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

[GitHub] [tvm] MasterJH5574 commented on pull request #9728: [BugFix] Fix a wrong use of `std::move()` in cross-thread reduction lowering

2021-12-13 Thread GitBox
MasterJH5574 commented on pull request #9728: URL: https://github.com/apache/tvm/pull/9728#issuecomment-992661562 :joy: Given the [undefined behavior of evaluation order](https://stackoverflow.com/questions/2934904/order-of-evaluation-in-c-function-parameters), we decided to remove

[GitHub] [tvm] manupa-arm commented on pull request #8472: [TIR] Introduce tir.allocate_const to TIR

2021-12-13 Thread GitBox
manupa-arm commented on pull request #8472: URL: https://github.com/apache/tvm/pull/8472#issuecomment-992685727 @d-smirnov I suppose this is the first PR that needs to go in ? (I m waiting to review this though I've seen updates to the PR that depends on this). Let me know what

[GitHub] [tvm] mbs-octoml commented on a change in pull request #9693: [Relay] PlanDevices supports 'free' on_device annotations

2021-12-13 Thread GitBox
mbs-octoml commented on a change in pull request #9693: URL: https://github.com/apache/tvm/pull/9693#discussion_r767968431 ## File path: include/tvm/target/se_scope.h ## @@ -170,7 +170,7 @@ class SEScopeNode : public AttrsNode { * * kInvalidDeviceType denotes

[GitHub] [tvm] mikepapadim opened a new pull request #9729: [CORE] Make external mods first class field in the IR module

2021-12-13 Thread GitBox
mikepapadim opened a new pull request #9729: URL: https://github.com/apache/tvm/pull/9729 Co-authored-by: Lily Orth-Smith This PR extracts the external modules used by the TECOmpiler and various places in the AOT flow and makes them first class fields in the IRModule.

[GitHub] [tvm] mbs-octoml commented on a change in pull request #9693: [Relay] PlanDevices supports 'free' on_device annotations

2021-12-13 Thread GitBox
mbs-octoml commented on a change in pull request #9693: URL: https://github.com/apache/tvm/pull/9693#discussion_r767974458 ## File path: src/relay/op/call/call.cc ## @@ -63,23 +63,23 @@ bool CallLoweredRel(const Array& types, int num_inputs, const Attrs& attrs const Op&

[GitHub] [tvm] mbs-octoml commented on a change in pull request #9693: [Relay] PlanDevices supports 'free' on_device annotations

2021-12-13 Thread GitBox
mbs-octoml commented on a change in pull request #9693: URL: https://github.com/apache/tvm/pull/9693#discussion_r768010547 ## File path: src/relay/transforms/device_planner.cc ## @@ -844,7 +932,8 @@ class DeviceCapturer : public ExprMutator { // match. return

[GitHub] [tvm] mbs-octoml commented on pull request #9693: [Relay] PlanDevices supports 'free' on_device annotations

2021-12-13 Thread GitBox
mbs-octoml commented on pull request #9693: URL: https://github.com/apache/tvm/pull/9693#issuecomment-992747464 Thanks @electriclilies and @jroesch , merged your post-submit comments into the sequel. -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [tvm] leandron commented on a change in pull request #9725: [microNPU] Update Arm(R) Ethos(TM)-U55 NPU demo README

2021-12-13 Thread GitBox
leandron commented on a change in pull request #9725: URL: https://github.com/apache/tvm/pull/9725#discussion_r767949380 ## File path: apps/microtvm/ethosu/README.md ## @@ -43,6 +43,10 @@ If the demo is not run in the ci_cpu Docker container, then you will need the fo You

[GitHub] [tvm] abdulazizm commented on issue #7990: LLVM ERROR: out of memory, when tracing maskrcnn model in deploy_object_detection_pytorch.py tutorial

2021-12-13 Thread GitBox
abdulazizm commented on issue #7990: URL: https://github.com/apache/tvm/issues/7990#issuecomment-992694906 @daniperfer sorry i am newbie, how can we change this llvm version? Is there any blog or documentation that you refer? -- This is an automated message from the Apache Git Service.

[GitHub] [tvm] mbs-octoml commented on a change in pull request #9693: [Relay] PlanDevices supports 'free' on_device annotations

2021-12-13 Thread GitBox
mbs-octoml commented on a change in pull request #9693: URL: https://github.com/apache/tvm/pull/9693#discussion_r767969023 ## File path: src/relay/transforms/device_planner.cc ## @@ -20,6 +20,10 @@ /*! * \file src/relay/transforms/device_planner.cc * \brief Determines a

[GitHub] [tvm] junrushao1994 commented on a change in pull request #9729: [CORE] Make external mods first class field in the IR module

2021-12-13 Thread GitBox
junrushao1994 commented on a change in pull request #9729: URL: https://github.com/apache/tvm/pull/9729#discussion_r767980273 ## File path: include/tvm/ir/module.h ## @@ -57,6 +57,8 @@ class IRModuleNode : public Object { Map functions; /*! \brief A map from global type

[GitHub] [tvm] comaniac commented on a change in pull request #9721: [Relay, BYOC] Make constant binding in PartitionGraph optional

2021-12-13 Thread GitBox
comaniac commented on a change in pull request #9721: URL: https://github.com/apache/tvm/pull/9721#discussion_r767983007 ## File path: python/tvm/relay/transform/transform.py ## @@ -695,17 +695,23 @@ def LambdaLift(): return _ffi_api.LambdaLift() -def

[GitHub] [tvm] junrushao1994 commented on pull request #9728: [BugFix] Fix a wrong use of `std::move()` in cross-thread reduction lowering

2021-12-13 Thread GitBox
junrushao1994 commented on pull request #9728: URL: https://github.com/apache/tvm/pull/9728#issuecomment-992658390 Looks like I triggered an undefined behavior of C++ 梁 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [tvm] mbs-octoml commented on a change in pull request #9669: [BugFix] shapeOfAttrs should be registered before "vm.shape_of" used

2021-12-13 Thread GitBox
mbs-octoml commented on a change in pull request #9669: URL: https://github.com/apache/tvm/pull/9669#discussion_r767960497 ## File path: tests/python/relay/test_vm.py ## @@ -1112,6 +1112,20 @@ def test_multi_targets(): tvm.testing.assert_allclose(actual_result.numpy(),

[GitHub] [tvm] mbs-octoml commented on a change in pull request #9693: [Relay] PlanDevices supports 'free' on_device annotations

2021-12-13 Thread GitBox
mbs-octoml commented on a change in pull request #9693: URL: https://github.com/apache/tvm/pull/9693#discussion_r767969790 ## File path: include/tvm/relay/attrs/on_device.h ## @@ -54,44 +54,48 @@ namespace relay { * multiply(device_copy(add(%x, %y), src_se_scope=GPU,

[GitHub] [tvm] mbs-octoml commented on a change in pull request #9693: [Relay] PlanDevices supports 'free' on_device annotations

2021-12-13 Thread GitBox
mbs-octoml commented on a change in pull request #9693: URL: https://github.com/apache/tvm/pull/9693#discussion_r767970440 ## File path: src/printer/text_printer.cc ## @@ -36,49 +37,71 @@ static const char* kSemVer = "0.0.5"; Doc TextPrinter::PrintMod(const IRModule& mod) {

[GitHub] [tvm] mbs-octoml commented on a change in pull request #9693: [Relay] PlanDevices supports 'free' on_device annotations

2021-12-13 Thread GitBox
mbs-octoml commented on a change in pull request #9693: URL: https://github.com/apache/tvm/pull/9693#discussion_r768004662 ## File path: src/relay/op/memory/on_device.cc ## @@ -45,53 +43,74 @@ const Op& OnDeviceOp() { return op; } -Expr OnDevice(Expr expr, SEScope

[GitHub] [tvm] mbs-octoml commented on a change in pull request #9693: [Relay] PlanDevices supports 'free' on_device annotations

2021-12-13 Thread GitBox
mbs-octoml commented on a change in pull request #9693: URL: https://github.com/apache/tvm/pull/9693#discussion_r768009277 ## File path: src/relay/op/memory/on_device.h ## @@ -39,14 +39,50 @@ namespace relay { const Op& OnDeviceOp(); /*! - * \brief Wraps \p expr in an

[GitHub] [tvm] leandron merged pull request #9722: [TVMC] Add --opt-level to compile mode

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

[tvm] branch main updated: [TVMC] Add --opt-level to compile mode (#9722)

2021-12-13 Thread leandron
This is an automated email from the ASF dual-hosted git repository. leandron 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 6e9e4e6 [TVMC] Add --opt-level to compile mode

[GitHub] [tvm] mbs-octoml commented on a change in pull request #9693: [Relay] PlanDevices supports 'free' on_device annotations

2021-12-13 Thread GitBox
mbs-octoml commented on a change in pull request #9693: URL: https://github.com/apache/tvm/pull/9693#discussion_r767972447 ## File path: src/printer/text_printer.cc ## @@ -36,49 +37,71 @@ static const char* kSemVer = "0.0.5"; Doc TextPrinter::PrintMod(const IRModule& mod) {

[GitHub] [tvm] junrushao1994 commented on a change in pull request #9729: [CORE] Make external mods first class field in the IR module

2021-12-13 Thread GitBox
junrushao1994 commented on a change in pull request #9729: URL: https://github.com/apache/tvm/pull/9729#discussion_r767980273 ## File path: include/tvm/ir/module.h ## @@ -57,6 +57,8 @@ class IRModuleNode : public Object { Map functions; /*! \brief A map from global type

[GitHub] [tvm] mbs-octoml commented on a change in pull request #9693: [Relay] PlanDevices supports 'free' on_device annotations

2021-12-13 Thread GitBox
mbs-octoml commented on a change in pull request #9693: URL: https://github.com/apache/tvm/pull/9693#discussion_r768005529 ## File path: src/relay/op/memory/on_device.cc ## @@ -45,53 +43,74 @@ const Op& OnDeviceOp() { return op; } -Expr OnDevice(Expr expr, SEScope

[GitHub] [tvm] comaniac commented on a change in pull request #9729: [CORE] Make external mods first class field in the IR module

2021-12-13 Thread GitBox
comaniac commented on a change in pull request #9729: URL: https://github.com/apache/tvm/pull/9729#discussion_r768008092 ## File path: include/tvm/ir/module.h ## @@ -57,6 +57,8 @@ class IRModuleNode : public Object { Map functions; /*! \brief A map from global type vars

[GitHub] [tvm] mbs-octoml commented on a change in pull request #9693: [Relay] PlanDevices supports 'free' on_device annotations

2021-12-13 Thread GitBox
mbs-octoml commented on a change in pull request #9693: URL: https://github.com/apache/tvm/pull/9693#discussion_r768007923 ## File path: src/relay/op/memory/on_device.cc ## @@ -45,53 +43,74 @@ const Op& OnDeviceOp() { return op; } -Expr OnDevice(Expr expr, SEScope

[GitHub] [tvm] electriclilies commented on a change in pull request #9729: [CORE] Make external mods first class field in the IR module

2021-12-13 Thread GitBox
electriclilies commented on a change in pull request #9729: URL: https://github.com/apache/tvm/pull/9729#discussion_r768016573 ## File path: include/tvm/ir/module.h ## @@ -57,6 +57,8 @@ class IRModuleNode : public Object { Map functions; /*! \brief A map from global type

[GitHub] [tvm-rfcs] areusch commented on pull request #46: Module Based Model Runtime for AOT

2021-12-13 Thread GitBox
areusch commented on pull request #46: URL: https://github.com/apache/tvm-rfcs/pull/46#issuecomment-992762313 @manupa-arm thanks! I think ultimately the @ashutosh-arm pass could be a good way to implement ConstLoader, but for now I'm hoping we can simply rename. In the long run, I'd

[GitHub] [tvm] junrushao1994 commented on a change in pull request #9729: [CORE] Make external mods first class field in the IR module

2021-12-13 Thread GitBox
junrushao1994 commented on a change in pull request #9729: URL: https://github.com/apache/tvm/pull/9729#discussion_r768036732 ## File path: include/tvm/ir/module.h ## @@ -57,6 +57,8 @@ class IRModuleNode : public Object { Map functions; /*! \brief A map from global type

[GitHub] [tvm] areusch commented on a change in pull request #9554: Implement [skip ci] for Jenkins

2021-12-13 Thread GitBox
areusch commented on a change in pull request #9554: URL: https://github.com/apache/tvm/pull/9554#discussion_r768041428 ## File path: tests/python/unittest/test_ci.py ## @@ -0,0 +1,116 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor

[GitHub] [tvm] Lunderberg commented on pull request #9727: [TE][TIR] Implement layout transformations, non-flat memory buffers

2021-12-13 Thread GitBox
Lunderberg commented on pull request #9727: URL: https://github.com/apache/tvm/pull/9727#issuecomment-992799674 Fixing lint issues. The large number of commits listed is due to rebasing the fixes into the commits that introduced them. -- This is an automated message from the Apache Git

[GitHub] [tvm] hzfan closed pull request #9720: [Relay] Non-recursive dependency graph

2021-12-13 Thread GitBox
hzfan closed pull request #9720: URL: https://github.com/apache/tvm/pull/9720 -- 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:

[GitHub] [tvm] jroesch commented on a change in pull request #9613: [Relay] Re-run PlanDevices after LowerTE to flow new memory scope constraints.

2021-12-13 Thread GitBox
jroesch commented on a change in pull request #9613: URL: https://github.com/apache/tvm/pull/9613#discussion_r768066039 ## File path: include/tvm/relay/expr.h ## @@ -234,6 +234,15 @@ class Var : public Expr { */ TVM_DLL Var(Id vid, Type type_annotation, Span span =

[GitHub] [tvm] comaniac commented on pull request #9528: [Relay] Non-recursive dependency graph

2021-12-13 Thread GitBox
comaniac commented on pull request #9528: URL: https://github.com/apache/tvm/pull/9528#issuecomment-992817707 Thanks @hzfan @mbrookhart -- 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

[GitHub] [tvm] comaniac merged pull request #9528: [Relay] Non-recursive dependency graph

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

[GitHub] [tvm] driazati commented on pull request #9696: Add CMake summary

2021-12-13 Thread GitBox
driazati commented on pull request #9696: URL: https://github.com/apache/tvm/pull/9696#issuecomment-992816914 It is now with https://github.com/apache/tvm/pull/9696/commits/e680e69d77ea9ed05d46fce701518578e21b45b0 ([test

[GitHub] [tvm] mbs-octoml commented on a change in pull request #9729: [CORE] Make external mods first class field in the IR module

2021-12-13 Thread GitBox
mbs-octoml commented on a change in pull request #9729: URL: https://github.com/apache/tvm/pull/9729#discussion_r768029280 ## File path: include/tvm/ir/module.h ## @@ -57,6 +57,8 @@ class IRModuleNode : public Object { Map functions; /*! \brief A map from global type

[GitHub] [tvm] driazati commented on a change in pull request #9554: Implement [skip ci] for Jenkins

2021-12-13 Thread GitBox
driazati commented on a change in pull request #9554: URL: https://github.com/apache/tvm/pull/9554#discussion_r768049498 ## File path: tests/python/unittest/test_ci.py ## @@ -0,0 +1,116 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor

[GitHub] [tvm] driazati commented on pull request #9554: Implement [skip ci] for Jenkins

2021-12-13 Thread GitBox
driazati commented on pull request #9554: URL: https://github.com/apache/tvm/pull/9554#issuecomment-992791257 Confirming sounds good, it should be ready to push now -- 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 (6e9e4e6 -> a674121)

2021-12-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 6e9e4e6 [TVMC] Add --opt-level to compile mode (#9722) add a674121 [Relay] Non-recursive dependency graph

[GitHub] [tvm] AndrewZhaoLuo opened a new pull request #9735: [WIP] Add faster type inference

2021-12-13 Thread GitBox
AndrewZhaoLuo opened a new pull request #9735: URL: https://github.com/apache/tvm/pull/9735 TODO -- 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:

[GitHub] [tvm] ccjoechou commented on pull request #9730: [pre-RFC][BYOC] Marvell ML/AI Accelerator Integration

2021-12-13 Thread GitBox
ccjoechou commented on pull request #9730: URL: https://github.com/apache/tvm/pull/9730#issuecomment-993059437 Hello @tqchen, @jroesch, or any other reviewer: We have also posted a pre-RFC

[GitHub] [tvm] ganler commented on a change in pull request #9582: [BugFix] resolve integer 32. ~ 64. mismatch by casting

2021-12-13 Thread GitBox
ganler commented on a change in pull request #9582: URL: https://github.com/apache/tvm/pull/9582#discussion_r757714983 ## File path: tests/python/relay/test_type_solver.py ## @@ -338,6 +341,34 @@ def test_incompatible_quantified_func_unification(): solver.Unify(ft1, ft2)

[tvm] branch main updated (f0ae9d3 -> 1728782)

2021-12-13 Thread jroesch
This is an automated email from the ASF dual-hosted git repository. jroesch pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/tvm.git. from f0ae9d3 Add `make docs` and doc building instructions (#9534) add 1728782 Fix typos in runtime comments (#9726)

[GitHub] [tvm] jroesch merged pull request #9726: Fix typos in runtime comments

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

[GitHub] [tvm] junrushao1994 merged pull request #9728: [BugFix] Fix a wrong use of `std::move()` in cross-thread reduction lowering

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

[tvm] branch main updated: [BugFix] Fix a wrong use of `std::move()` in cross-thread reduction lowering (#9728)

2021-12-13 Thread junrushao
This is an automated email from the ASF dual-hosted git repository. junrushao 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 4e70931 [BugFix] Fix a wrong use of `std::move()`

[GitHub] [tvm] CircleSpin commented on a change in pull request #9633: Tvmc python tutorial

2021-12-13 Thread GitBox
CircleSpin commented on a change in pull request #9633: URL: https://github.com/apache/tvm/pull/9633#discussion_r768268601 ## File path: gallery/how_to/use_tvms_python_api/tvmc_python.py ## @@ -0,0 +1,252 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or

[GitHub] [tvm] CircleSpin commented on pull request #9633: Tvmc python tutorial

2021-12-13 Thread GitBox
CircleSpin commented on pull request #9633: URL: https://github.com/apache/tvm/pull/9633#issuecomment-993095492 Hi @hogepodge, By all means if there's another ordering of the tutorials you prefer let's go with that. This tutorial seems to do the same thing as this one

[tvm] branch main updated (69cae0b -> b4d595c)

2021-12-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 69cae0b [Relay, BYOC] Make constant binding in PartitionGraph optional (#9721) add b4d595c Add CMake summary

[GitHub] [tvm] zhiics commented on pull request #9729: [CORE] Make external mods first class field in the IR module

2021-12-13 Thread GitBox
zhiics commented on pull request #9729: URL: https://github.com/apache/tvm/pull/9729#issuecomment-993172582 @mikepapadim Thanks for the PR. Could you elaborate a bit the motivation of moving the external runtime module into the IRModule? IMOH, the IRModule, particularly the relay IR here,

[GitHub] [tvm] masahi closed issue #9653: [Bug] test_tensorrt.py::test_conv2d_transpose failed

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

[tvm] branch main updated (b4d595c -> 1f5f3c9)

2021-12-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 b4d595c Add CMake summary (#9696) add 1f5f3c9 [CUDA] Do not emit vector load on unaligned base offset (#9731) No

[GitHub] [tvm] areusch merged pull request #9534: Add `make docs` and doc building instructions

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

[tvm] branch main updated (a674121 -> f0ae9d3)

2021-12-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 a674121 [Relay] Non-recursive dependency graph (#9528) add f0ae9d3 Add `make docs` and doc building instructions

[GitHub] [tvm] masahi merged pull request #9698: [CUTLASS] More robust support for pattern matching and alignment

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

[GitHub] [tvm] masahi merged pull request #9731: [CUDA] Do not emit vector load on unaligned base offset

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

[GitHub] [tvm] masahi closed issue #8540: [BUG] CUDA misaligned address when vectorizing float16

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

[GitHub] [tvm] jroesch commented on pull request #9730: [pre-RFC][BYOC] Marvell ML/AI Accelerator Integration

2021-12-13 Thread GitBox
jroesch commented on pull request #9730: URL: https://github.com/apache/tvm/pull/9730#issuecomment-993048649 Thanks for your contribution. When you have a chance could you update the description of the PR with more details, and potentially link to the pre-RFC discussion? Thanks! --

[tvm] branch main updated (4e70931 -> d1dafbd)

2021-12-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 4e70931 [BugFix] Fix a wrong use of `std::move()` in cross-thread reduction lowering (#9728) add d1dafbd

  1   2   >