[tvm] branch main updated (1d3177b -> d9a5ff5)

2021-10-04 Thread junrushao
This is an automated email from the ASF dual-hosted git repository. junrushao pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/tvm.git. from 1d3177b [TIR] Fix FlattenBuffer computing size for buffer with strides (#9195) add d9a5ff5 Update ci-cpu to v0.

[GitHub] [tvm] junrushao1994 closed issue #9158: [CI Image] Update ci_i386 to 20210929-225912-da5b4fbfe

2021-10-04 Thread GitBox
junrushao1994 closed issue #9158: URL: https://github.com/apache/tvm/issues/9158 -- 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] junrushao1994 merged pull request #9199: Update ci-cpu to v0.78

2021-10-04 Thread GitBox
junrushao1994 merged pull request #9199: URL: https://github.com/apache/tvm/pull/9199 -- 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-unsu

[tvm] branch main updated (7c6d71b -> 1d3177b)

2021-10-04 Thread junrushao
This is an automated email from the ASF dual-hosted git repository. junrushao pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/tvm.git. from 7c6d71b [UnitTests][CMSISNN] Mark CMSISNN with skipif they are missing libraries (#9179) add 1d3177b [TIR] Fix

[GitHub] [tvm] junrushao1994 merged pull request #9195: [TIR] Fix FlattenBuffer computing size for buffer with strides

2021-10-04 Thread GitBox
junrushao1994 merged pull request #9195: URL: https://github.com/apache/tvm/pull/9195 -- 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-unsu

[GitHub] [tvm] huajsj commented on a change in pull request #9108: [Runtime] Pipeline Executor Second patch, configuration load and executor export/import.

2021-10-04 Thread GitBox
huajsj commented on a change in pull request #9108: URL: https://github.com/apache/tvm/pull/9108#discussion_r721937140 ## File path: src/runtime/pipeline/pipeline_function.h ## @@ -0,0 +1,60 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more contr

[GitHub] [tvm] huajsj commented on a change in pull request #9108: [Runtime] Pipeline Executor Second patch, configuration load and executor export/import.

2021-10-04 Thread GitBox
huajsj commented on a change in pull request #9108: URL: https://github.com/apache/tvm/pull/9108#discussion_r721934740 ## File path: src/runtime/pipeline/pipeline_function.cc ## @@ -0,0 +1,101 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more con

[GitHub] [tvm] jiangjiajun commented on pull request #9083: Add TVMC Frontend for PaddlePaddle

2021-10-04 Thread GitBox
jiangjiajun commented on pull request #9083: URL: https://github.com/apache/tvm/pull/9083#issuecomment-934114836 @leandron Sorry, I was handling another PR last week, but I noticed there's a request for updating the ci image, and already passed the testing. After https://github.com/apache

[GitHub] [tvm] huajsj commented on a change in pull request #9108: [Runtime] Pipeline Executor Second patch, configuration load and executor export/import.

2021-10-04 Thread GitBox
huajsj commented on a change in pull request #9108: URL: https://github.com/apache/tvm/pull/9108#discussion_r721933359 ## File path: src/runtime/pipeline/pipeline_executor.h ## @@ -36,25 +43,89 @@ namespace runtime { * * This executor can be accessed by various language vi

[GitHub] [tvm] huajsj commented on a change in pull request #9108: [Runtime] Pipeline Executor Second patch, configuration load and executor export/import.

2021-10-04 Thread GitBox
huajsj commented on a change in pull request #9108: URL: https://github.com/apache/tvm/pull/9108#discussion_r721928615 ## File path: src/runtime/pipeline/pipeline_executor.cc ## @@ -21,31 +21,48 @@ * \file pipeline_executor.cc */ #include "pipeline_executor.h" - namespace

[GitHub] [tvm] huajsj commented on a change in pull request #9108: [Runtime] Pipeline Executor Second patch, configuration load and executor export/import.

2021-10-04 Thread GitBox
huajsj commented on a change in pull request #9108: URL: https://github.com/apache/tvm/pull/9108#discussion_r721912249 ## File path: python/tvm/contrib/pipeline_executor.py ## @@ -501,17 +538,18 @@ class PipelineExecutorFactoryModule(object): """ -def __init__(self

[GitHub] [tvm] jiangjiajun commented on a change in pull request #9126: [Frontend][PaddlePaddle][Part1] Add 100+ operators for PaddlePaddle

2021-10-04 Thread GitBox
jiangjiajun commented on a change in pull request #9126: URL: https://github.com/apache/tvm/pull/9126#discussion_r721907908 ## File path: tests/python/frontend/paddlepaddle/test_forward.py ## @@ -623,39 +829,30 @@ def slice4(inputs): @tvm.testing.uses_gpu -def test_forward

[GitHub] [tvm] huajsj edited a comment on pull request #9108: [Runtime] Pipeline Executor Second patch, configuration load and executor export/import.

2021-10-04 Thread GitBox
huajsj edited a comment on pull request #9108: URL: https://github.com/apache/tvm/pull/9108#issuecomment-934084988 > @huajsj Why `mod_index` etc starts from 1? What does index 0 corresponds to? Unless there is a good reason for this weird design, please make all indices start from 0.

[GitHub] [tvm] huajsj commented on pull request #9108: [Runtime] Pipeline Executor Second patch, configuration load and executor export/import.

2021-10-04 Thread GitBox
huajsj commented on pull request #9108: URL: https://github.com/apache/tvm/pull/9108#issuecomment-934084988 > @huajsj Why `mod_index` etc starts from 1? What does index 0 corresponds to? Unless there is a good reason for this weird design, please make all indices start from 0. @masa

[GitHub] [tvm] huajsj commented on a change in pull request #9108: [Runtime] Pipeline Executor Second patch, configuration load and executor export/import.

2021-10-04 Thread GitBox
huajsj commented on a change in pull request #9108: URL: https://github.com/apache/tvm/pull/9108#discussion_r721903589 ## File path: src/runtime/pipeline/pipeline_struct.h ## @@ -0,0 +1,170 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more contri

[GitHub] [tvm] abhikran-quic commented on a change in pull request #9186: [ONNX] Add MatMulInteger16 contrib op

2021-10-04 Thread GitBox
abhikran-quic commented on a change in pull request #9186: URL: https://github.com/apache/tvm/pull/9186#discussion_r721892645 ## File path: tests/python/frontend/onnx/test_forward.py ## @@ -1282,6 +1282,45 @@ def verify_batch_matmul(a_shape, b_shape, out_shape, convert_config=

[GitHub] [tvm] huajsj commented on a change in pull request #9108: [Runtime] Pipeline Executor Second patch, configuration load and executor export/import.

2021-10-04 Thread GitBox
huajsj commented on a change in pull request #9108: URL: https://github.com/apache/tvm/pull/9108#discussion_r721874961 ## File path: src/runtime/pipeline/pipeline_struct.h ## @@ -0,0 +1,170 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more contri

[GitHub] [tvm] huajsj commented on a change in pull request #9108: [Runtime] Pipeline Executor Second patch, configuration load and executor export/import.

2021-10-04 Thread GitBox
huajsj commented on a change in pull request #9108: URL: https://github.com/apache/tvm/pull/9108#discussion_r721872538 ## File path: src/runtime/pipeline/pipeline_function.h ## @@ -0,0 +1,60 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more contr

[tvm] branch main updated: [UnitTests][CMSISNN] Mark CMSISNN with skipif they are missing libraries (#9179)

2021-10-04 Thread masahi
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 7c6d71b [UnitTests][CMSISNN] Mark CMSISNN with skipif

[tvm] branch main updated: [Meta Scheduler] Add cleanup for localrunner (#9191)

2021-10-04 Thread masahi
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 86b184b [Meta Scheduler] Add cleanup for localrunner (

[GitHub] [tvm] masahi merged pull request #9179: [UnitTests][CMSISNN] Mark CMSISNN with skipif they are missing libraries

2021-10-04 Thread GitBox
masahi merged pull request #9179: URL: https://github.com/apache/tvm/pull/9179 -- 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 (6911339 -> 16cabb7)

2021-10-04 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 6911339 correct error (#9193) add 16cabb7 Fix custom_address serialization in c++ tracker client. (#9192) No new

[GitHub] [tvm] masahi merged pull request #9191: [Meta Scheduler] Add cleanup for localrunner

2021-10-04 Thread GitBox
masahi merged pull request #9191: URL: https://github.com/apache/tvm/pull/9191 -- 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] masahi merged pull request #9192: Fix custom_address serialization in c++ tracker client.

2021-10-04 Thread GitBox
masahi merged pull request #9192: URL: https://github.com/apache/tvm/pull/9192 -- 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 (a6be791 -> 6911339)

2021-10-04 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 a6be791 Add USE_ETHOSU for the config.cmake (#9162) add 6911339 correct error (#9193) No new revisions were added

[GitHub] [tvm] masahi merged pull request #9193: [Typo] Correct fast_tanh description

2021-10-04 Thread GitBox
masahi merged pull request #9193: URL: https://github.com/apache/tvm/pull/9193 -- 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] vinx13 commented on a change in pull request #9168: Adding annotations for tir.allocate

2021-10-04 Thread GitBox
vinx13 commented on a change in pull request #9168: URL: https://github.com/apache/tvm/pull/9168#discussion_r721830600 ## File path: include/tvm/tir/stmt.h ## @@ -817,9 +827,7 @@ class ForNode : public StmtNode { * \brief Additional annotations about the loop. * *

[GitHub] [tvm] areusch commented on issue #9158: [CI Image] Update ci_i386 to 20210929-225912-da5b4fbfe

2021-10-04 Thread GitBox
areusch commented on issue #9158: URL: https://github.com/apache/tvm/issues/9158#issuecomment-933966583 ``` $ docker tag tlcpackstaging/ci_cpu:20210929-225912-da5b4fbfe-validated tlcpack/ci-cpu:v0.78 $ docker push tlcpack/ci-cpu:v0.78 ``` Also created PR tagged above. --

[GitHub] [tvm] areusch opened a new pull request #9199: Update ci-cpu to v0.78

2021-10-04 Thread GitBox
areusch opened a new pull request #9199: URL: https://github.com/apache/tvm/pull/9199 * Close #9158 cc @Mousius @jtuyls -- 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 comm

[tvm] 01/01: Update ci-cpu to v0.78.

2021-10-04 Thread areusch
This is an automated email from the ASF dual-hosted git repository. areusch pushed a commit to branch ci-docker-staging in repository https://gitbox.apache.org/repos/asf/tvm.git commit 007443dc40b415f82bb2fe65d8960c623df75491 Author: Andrew Reusch AuthorDate: Mon Oct 4 17:40:17 2021 -0700 U

[tvm] branch ci-docker-staging updated (e108fcf -> 007443d)

2021-10-04 Thread areusch
This is an automated email from the ASF dual-hosted git repository. areusch pushed a change to branch ci-docker-staging in repository https://gitbox.apache.org/repos/asf/tvm.git. discard e108fcf Fix typo in ci_cpu commands omit f8d8f5e [CI] Split Integration tests out of first phase of pi

[GitHub] [tvm] areusch commented on a change in pull request #9175: Fix end to end benchmark with rpc devices

2021-10-04 Thread GitBox
areusch commented on a change in pull request #9175: URL: https://github.com/apache/tvm/pull/9175#discussion_r721803719 ## File path: tests/python/relay/test_backend_graph_executor.py ## @@ -361,21 +361,21 @@ def test_benchmark_end_to_end(dev, target): assert len(result.re

[GitHub] [tvm] areusch commented on pull request #9190: Initial Implementation of TIRToRuntime Target hook

2021-10-04 Thread GitBox
areusch commented on pull request #9190: URL: https://github.com/apache/tvm/pull/9190#issuecomment-933957506 cc @mbs-octoml @electriclilies @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 t

[tvm] branch main updated (bab871a -> a6be791)

2021-10-04 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 bab871a [Hexagon] Fix compilation errors in Hexagon launcher (#9189) add a6be791 Add USE_ETHOSU for the config.cma

[GitHub] [tvm] areusch merged pull request #9162: Add USE_ETHOSU for the config.cmake

2021-10-04 Thread GitBox
areusch merged pull request #9162: URL: https://github.com/apache/tvm/pull/9162 -- 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 commented on pull request #9129: [CI] Pass failFast flag to Jenkins parallel

2021-10-04 Thread GitBox
areusch commented on pull request #9129: URL: https://github.com/apache/tvm/pull/9129#issuecomment-933956298 @Mousius > I do empathise with this, but I don't think we should design a CI solution around the edge cases, by reducing the overall running jobs we can get to these faster when

[tvm] branch main updated (6f98e2d -> bab871a)

2021-10-04 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 6f98e2d [CMSIS-NN] Initial operator support for Add (#9167) add bab871a [Hexagon] Fix compilation errors in Hexago

[GitHub] [tvm] areusch commented on pull request #9189: [Hexagon] Fix compilation errors in Hexagon launcher

2021-10-04 Thread GitBox
areusch commented on pull request #9189: URL: https://github.com/apache/tvm/pull/9189#issuecomment-933947363 thanks @kparzysz-quic ! -- 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] areusch merged pull request #9189: [Hexagon] Fix compilation errors in Hexagon launcher

2021-10-04 Thread GitBox
areusch merged pull request #9189: URL: https://github.com/apache/tvm/pull/9189 -- 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-rfcs] areusch commented on pull request #9: [RFC] TVM Unified Static Memory Planning

2021-10-04 Thread GitBox
areusch commented on pull request #9: URL: https://github.com/apache/tvm-rfcs/pull/9#issuecomment-933941308 @manupa-arm please open a tracking issue for this one. excited to see this land! -- This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [tvm-rfcs] areusch merged pull request #9: [RFC] TVM Unified Static Memory Planning

2021-10-04 Thread GitBox
areusch merged pull request #9: URL: https://github.com/apache/tvm-rfcs/pull/9 -- 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-rfcs] branch main updated: [RFC] TVM Unified Static Memory Planning (#9)

2021-10-04 Thread areusch
This is an automated email from the ASF dual-hosted git repository. areusch pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/tvm-rfcs.git The following commit(s) were added to refs/heads/main by this push: new 3834d72 [RFC] TVM Unified Static Memory Planning

[GitHub] [tvm] tmoreau89 commented on issue #8838: [Tracking Issue][ONNX] Quantized operator support in ONNX importer

2021-10-04 Thread GitBox
tmoreau89 commented on issue #8838: URL: https://github.com/apache/tvm/issues/8838#issuecomment-933939591 Thanks @abhikran-quic , added -- 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 specif

[GitHub] [tvm] areusch commented on pull request #8922: Arm(R) Cortex(R)-M55 CPU and Arm(R) Ethos(TM)-U55 NPU Demo App

2021-10-04 Thread GitBox
areusch commented on pull request #8922: URL: https://github.com/apache/tvm/pull/8922#issuecomment-933939581 neat! :) would be interested to discuss about perhaps creating apps/microtvm/ethosu e.g. project API impl. also have one question about the Python deps. -- This is an automated m

[GitHub] [tvm] areusch commented on a change in pull request #8922: Arm(R) Cortex(R)-M55 CPU and Arm(R) Ethos(TM)-U55 NPU Demo App

2021-10-04 Thread GitBox
areusch commented on a change in pull request #8922: URL: https://github.com/apache/tvm/pull/8922#discussion_r721789482 ## File path: apps/microtvm/ethosu/requirements.txt ## @@ -0,0 +1,22 @@ +attrs>=21.2.0 Review comment: could you comment where you get the list of pac

[tvm] branch main updated (779a506 -> 6f98e2d)

2021-10-04 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 779a506 [Relay] Remove DeviceMap from LowerTE (#8788) add 6f98e2d [CMSIS-NN] Initial operator support for Add (#91

[GitHub] [tvm] areusch merged pull request #9167: [CMSIS-NN] Initial operator support for Add

2021-10-04 Thread GitBox
areusch merged pull request #9167: URL: https://github.com/apache/tvm/pull/9167 -- 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] adstraw opened a new pull request #9198: Hexagon conv2d full output slice

2021-10-04 Thread GitBox
adstraw opened a new pull request #9198: URL: https://github.com/apache/tvm/pull/9198 Adjust the Hexagon conv2d schedule to calculate a "full output slice" width-wise and channel-wise across the input and output. Adds optional `h_split_factor` which optionally splits the height `h` dimens

[GitHub] [tvm] areusch commented on pull request #9106: Migrate C Interface API Generation to C++

2021-10-04 Thread GitBox
areusch commented on pull request #9106: URL: https://github.com/apache/tvm/pull/9106#issuecomment-933934397 @Mousius lgtm but please address the CI failure -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abov

[GitHub] [tvm] tqchen commented on a change in pull request #9196: Documentation Refactor

2021-10-04 Thread GitBox
tqchen commented on a change in pull request #9196: URL: https://github.com/apache/tvm/pull/9196#discussion_r721778544 ## File path: docs/get_started/contribute/community.rst ## @@ -0,0 +1,96 @@ +.. Licensed to the Apache Software Foundation (ASF) under one Review comment:

[GitHub] [tvm] tqchen commented on a change in pull request #9196: Documentation Refactor

2021-10-04 Thread GitBox
tqchen commented on a change in pull request #9196: URL: https://github.com/apache/tvm/pull/9196#discussion_r721768811 ## File path: docs/get_started/contribute/community.rst ## @@ -0,0 +1,96 @@ +.. Licensed to the Apache Software Foundation (ASF) under one Review comment:

[GitHub] [tvm] tqchen commented on a change in pull request #9196: Documentation Refactor

2021-10-04 Thread GitBox
tqchen commented on a change in pull request #9196: URL: https://github.com/apache/tvm/pull/9196#discussion_r721778544 ## File path: docs/get_started/contribute/community.rst ## @@ -0,0 +1,96 @@ +.. Licensed to the Apache Software Foundation (ASF) under one Review comment:

[GitHub] [tvm] areusch commented on a change in pull request #9196: Documentation Refactor

2021-10-04 Thread GitBox
areusch commented on a change in pull request #9196: URL: https://github.com/apache/tvm/pull/9196#discussion_r721778212 ## File path: docs/get_started/contribute/community.rst ## @@ -0,0 +1,96 @@ +.. Licensed to the Apache Software Foundation (ASF) under one Review comment:

[GitHub] [tvm] hogepodge closed pull request #9196: Documentation Refactor

2021-10-04 Thread GitBox
hogepodge closed pull request #9196: URL: https://github.com/apache/tvm/pull/9196 -- 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] tqchen commented on a change in pull request #9196: Documentation Refactor

2021-10-04 Thread GitBox
tqchen commented on a change in pull request #9196: URL: https://github.com/apache/tvm/pull/9196#discussion_r721768811 ## File path: docs/get_started/contribute/community.rst ## @@ -0,0 +1,96 @@ +.. Licensed to the Apache Software Foundation (ASF) under one Review comment:

[GitHub] [tvm] hogepodge commented on pull request #9196: Documentation Refactor

2021-10-04 Thread GitBox
hogepodge commented on pull request #9196: URL: https://github.com/apache/tvm/pull/9196#issuecomment-933913743 That will be a significant lift, but I can restart the patch if it's necessary. -- This is an automated message from the Apache Git Service. To respond to the message, please lo

[GitHub] [tvm] areusch commented on pull request #8668: Visualization of Relay IR

2021-10-04 Thread GitBox
areusch commented on pull request #8668: URL: https://github.com/apache/tvm/pull/8668#issuecomment-933913528 thanks @chiwwang ! ping me when it's ready for review -- 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] areusch commented on a change in pull request #9163: [CMSIS-NN] Initial operator support for Mul

2021-10-04 Thread GitBox
areusch commented on a change in pull request #9163: URL: https://github.com/apache/tvm/pull/9163#discussion_r721767316 ## File path: tests/python/contrib/test_cmsisnn/utils.py ## @@ -0,0 +1,83 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contribu

[GitHub] [tvm] areusch commented on a change in pull request #9163: [CMSIS-NN] Initial operator support for Mul

2021-10-04 Thread GitBox
areusch commented on a change in pull request #9163: URL: https://github.com/apache/tvm/pull/9163#discussion_r721767184 ## File path: tests/python/contrib/test_cmsisnn/utils.py ## @@ -0,0 +1,83 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contribu

[GitHub] [tvm] tqchen edited a comment on pull request #9196: Documentation Refactor

2021-10-04 Thread GitBox
tqchen edited a comment on pull request #9196: URL: https://github.com/apache/tvm/pull/9196#issuecomment-933906489 Thanks @hogepodge for pushing this. - Just noticed that there are files that are moved and lost history(e.g. For example, `device_target_interactions.rst`). It would b

[GitHub] [tvm] tqchen edited a comment on pull request #9196: Documentation Refactor

2021-10-04 Thread GitBox
tqchen edited a comment on pull request #9196: URL: https://github.com/apache/tvm/pull/9196#issuecomment-933906489 Thanks @hogepodge for pushing this. A few things to note: - I noticed that there are files that are moved and lost history(e.g. For example, `device_target_interaction

[GitHub] [tvm] areusch commented on a change in pull request #9147: A follow up PR for 5/6 of Arm(R) Ethos(TM)-U NPU codegen

2021-10-04 Thread GitBox
areusch commented on a change in pull request #9147: URL: https://github.com/apache/tvm/pull/9147#discussion_r721765377 ## File path: tests/python/contrib/test_ethosu/test_tir_to_cs_translator.py ## @@ -764,7 +764,4 @@ def check_buffer(address, region, length, buffer_var):

[GitHub] [tvm] areusch commented on a change in pull request #9147: A follow up PR for 5/6 of Arm(R) Ethos(TM)-U NPU codegen

2021-10-04 Thread GitBox
areusch commented on a change in pull request #9147: URL: https://github.com/apache/tvm/pull/9147#discussion_r721765070 ## File path: tests/python/contrib/test_ethosu/test_tir_to_cs_translator.py ## @@ -764,7 +764,4 @@ def check_buffer(address, region, length, buffer_var):

[GitHub] [tvm] areusch commented on pull request #9168: Adding annotations for tir.allocate

2021-10-04 Thread GitBox
areusch commented on pull request #9168: URL: https://github.com/apache/tvm/pull/9168#issuecomment-933908410 please tag the tracking issue @manupa-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

[GitHub] [tvm] tqchen edited a comment on pull request #9196: Documentation Refactor

2021-10-04 Thread GitBox
tqchen edited a comment on pull request #9196: URL: https://github.com/apache/tvm/pull/9196#issuecomment-933906489 Thanks @hogepodge . I noticed that there are files that are moved and lost history(e.g. For example, `device_target_interactions.rst`). There are also files that get editor a

[GitHub] [tvm] tqchen edited a comment on pull request #9196: Documentation Refactor

2021-10-04 Thread GitBox
tqchen edited a comment on pull request #9196: URL: https://github.com/apache/tvm/pull/9196#issuecomment-933906489 Thanks @hogepodge . I noticed that there are files that are moved and lost history. For example, `device_target_interactions.rst`. It would be great to keep all the hist

[GitHub] [tvm] csullivan commented on a change in pull request #9138: [LLVM] Make changes needed for opaque pointers

2021-10-04 Thread GitBox
csullivan commented on a change in pull request #9138: URL: https://github.com/apache/tvm/pull/9138#discussion_r721761938 ## File path: src/target/llvm/codegen_llvm.cc ## @@ -861,10 +871,10 @@ llvm::Value* CodeGenLLVM::CreateIntrinsic(const CallNode* op) {

[GitHub] [tvm] tqchen commented on pull request #9196: Documentation Refactor

2021-10-04 Thread GitBox
tqchen commented on pull request #9196: URL: https://github.com/apache/tvm/pull/9196#issuecomment-933906489 Thanks @hogepodge . I noticed that there are files that are moved and lost history. For example, `device_target_interactions.rst`(originally created by @zxybazh ). It would be

[GitHub] [tvm] csullivan commented on a change in pull request #9138: [LLVM] Make changes needed for opaque pointers

2021-10-04 Thread GitBox
csullivan commented on a change in pull request #9138: URL: https://github.com/apache/tvm/pull/9138#discussion_r721761938 ## File path: src/target/llvm/codegen_llvm.cc ## @@ -861,10 +871,10 @@ llvm::Value* CodeGenLLVM::CreateIntrinsic(const CallNode* op) {

[GitHub] [tvm-rfcs] areusch commented on a change in pull request #38: [RFC] Improved multi-target handling

2021-10-04 Thread GitBox
areusch commented on a change in pull request #38: URL: https://github.com/apache/tvm-rfcs/pull/38#discussion_r721652382 ## File path: rfcs/00xx-improved-multi-target-handling.md ## @@ -0,0 +1,176 @@ +- Feature Name: improved-multi-target-handling +- Start Date: 2021-09-20 +- R

[GitHub] [tvm] hogepodge commented on issue #8987: [Tracking Issue] Formalize TVM Documentation Organization

2021-10-04 Thread GitBox
hogepodge commented on issue #8987: URL: https://github.com/apache/tvm/issues/8987#issuecomment-933902807 Pull request: https://github.com/apache/tvm/pull/9196 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the U

[GitHub] [tvm] AndrewZhaoLuo commented on a change in pull request #9126: [Frontend][PaddlePaddle][Part1] Add 100+ operators for PaddlePaddle

2021-10-04 Thread GitBox
AndrewZhaoLuo commented on a change in pull request #9126: URL: https://github.com/apache/tvm/pull/9126#discussion_r721756932 ## File path: tests/python/frontend/paddlepaddle/test_forward.py ## @@ -623,39 +829,30 @@ def slice4(inputs): @tvm.testing.uses_gpu -def test_forwa

[GitHub] [tvm] mbs-octoml opened a new pull request #9197: [Relay] Address Christopher's comments from #8788

2021-10-04 Thread GitBox
mbs-octoml opened a new pull request #9197: URL: https://github.com/apache/tvm/pull/9197 We don't need the Optional on ToANormalForm and friends. -- 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] tkonolige commented on pull request #9175: Fix end to end benchmark with rpc devices

2021-10-04 Thread GitBox
tkonolige commented on pull request #9175: URL: https://github.com/apache/tvm/pull/9175#issuecomment-933894413 @areusch or @junrushao1994 Can you review? -- 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 t

[GitHub] [tvm] hogepodge commented on pull request #9196: Documentation Refactor

2021-10-04 Thread GitBox
hogepodge commented on pull request #9196: URL: https://github.com/apache/tvm/pull/9196#issuecomment-933892463 A preview of the documentation is available at http://mochi.horse:8081 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Gi

[GitHub] [tvm] hogepodge opened a new pull request #9196: Documentation Refactor

2021-10-04 Thread GitBox
hogepodge opened a new pull request #9196: URL: https://github.com/apache/tvm/pull/9196 This patch is the documentation refactor described in RFC 0027: RFC: https://github.com/apache/tvm-rfcs/blob/main/rfcs/0027-formalize-documentation-organization.md Tracking Issue: https://github.co

[GitHub] [tvm] tmoreau89 commented on pull request #9194: [TIR] 1-d AllocateNode::extent

2021-10-04 Thread GitBox
tmoreau89 commented on pull request #9194: URL: https://github.com/apache/tvm/pull/9194#issuecomment-933832456 CC @kparzysz-quic -- 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 com

[GitHub] [tvm] Lunderberg commented on pull request #9179: [UnitTests][CMSISNN] Mark CMSISNN with skipif they are missing libraries

2021-10-04 Thread GitBox
Lunderberg commented on pull request #9179: URL: https://github.com/apache/tvm/pull/9179#issuecomment-933824827 We can, yes. My initial goal was to avoid adding too many utility functions that are only used for one or two tests, but I can see how having separate ways of doing the same thi

[GitHub] [tvm] Lunderberg commented on pull request #9194: [TIR] 1-d AllocateNode::extent

2021-10-04 Thread GitBox
Lunderberg commented on pull request #9194: URL: https://github.com/apache/tvm/pull/9194#issuecomment-933820926 @csullivan @tmoreau89 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specifi

[GitHub] [tvm] Lunderberg opened a new pull request #9194: [TIR] 1-d AllocateNode::extent

2021-10-04 Thread GitBox
Lunderberg opened a new pull request #9194: URL: https://github.com/apache/tvm/pull/9194 This PR replace the `AllocateNode::extents` variable, which represented the logical N-d shape of the tensor, with a 1-d `AllocateNode::extent` variable representing the amount of memory to be allocated

[GitHub] [tvm] mbs-octoml commented on a change in pull request #9159: Address review comments on Arm(R) Ethos(TM)-U PR 3/6

2021-10-04 Thread GitBox
mbs-octoml commented on a change in pull request #9159: URL: https://github.com/apache/tvm/pull/9159#discussion_r721674546 ## File path: python/tvm/relay/backend/contrib/ethosu/tir_to_cs_translator.py ## @@ -447,81 +455,75 @@ def _create_npu_feature_map(serial_feature_map):

[GitHub] [tvm] mbs-octoml commented on pull request #8788: [Relay] Remove DeviceMap from LowerTE

2021-10-04 Thread GitBox
mbs-octoml commented on pull request #8788: URL: https://github.com/apache/tvm/pull/8788#issuecomment-933801228 Thanks @jroesch. @Mousius I didn't push the changes for your comments before the merge but will get them in somehow. -- This is an automated message from the Apache Git Servic

[GitHub] [tvm] comaniac commented on a change in pull request #9108: [Runtime] Pipeline Executor Second patch, configuration load and executor export/import.

2021-10-04 Thread GitBox
comaniac commented on a change in pull request #9108: URL: https://github.com/apache/tvm/pull/9108#discussion_r721635064 ## File path: python/tvm/contrib/pipeline_executor.py ## @@ -86,8 +88,43 @@ class PipelineModule(object): Common interface for pipeline executor fac

[tvm] branch main updated (2f02b1e -> 779a506)

2021-10-04 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 2f02b1e support Torch all and any op (#9185) add 779a506 [Relay] Remove DeviceMap from LowerTE (#8788) No new rev

[GitHub] [tvm] jroesch merged pull request #8788: [Relay] Remove DeviceMap from LowerTE

2021-10-04 Thread GitBox
jroesch merged pull request #8788: URL: https://github.com/apache/tvm/pull/8788 -- 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] jroesch commented on pull request #8788: [Relay] Remove DeviceMap from LowerTE

2021-10-04 Thread GitBox
jroesch commented on pull request #8788: URL: https://github.com/apache/tvm/pull/8788#issuecomment-933780439 I am going to land this for sake of future pain, feel free to follow up if any other changes need to happen after this. -- This is an automated message from the Apache Git Servic

[GitHub] [tvm] jroesch commented on a change in pull request #8788: [Relay] Remove DeviceMap from LowerTE

2021-10-04 Thread GitBox
jroesch commented on a change in pull request #8788: URL: https://github.com/apache/tvm/pull/8788#discussion_r721646717 ## File path: include/tvm/relay/transform.h ## @@ -165,11 +165,12 @@ TVM_DLL Pass ToANormalForm(); /*! * \brief ToANormalForm but on incomplete graph. *

[GitHub] [tvm] rkimball commented on issue #9174: [Bug] [CPP RPC] custom_addr not properly quoted in JSON string

2021-10-04 Thread GitBox
rkimball commented on issue #9174: URL: https://github.com/apache/tvm/issues/9174#issuecomment-933767531 Here is a quick fix https://github.com/apache/tvm/pull/9192 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[GitHub] [tvm] rkimball opened a new pull request #9192: Fix custom_address serialization in c++ tracker client.

2021-10-04 Thread GitBox
rkimball opened a new pull request #9192: URL: https://github.com/apache/tvm/pull/9192 Make sure that if a custom address is passed then it is json serialized surrounded by quotes. Thanks for contributing to TVM! Please refer to guideline https://tvm.apache.org/docs/contribute/ fo

[GitHub] [tvm] areusch commented on pull request #9169: [Relay][VM][RPC]Use a uint64_t to serialize primitive_attrs in the Relay VM to fix 32bit RPC

2021-10-04 Thread GitBox
areusch commented on pull request #9169: URL: https://github.com/apache/tvm/pull/9169#issuecomment-933756394 hmm it might be possible to write a future test using ci-i386. cc @denise-k could you capture a tracking issue to think about this? -- This is an automated message from the Apache

[GitHub] [tvm-rfcs] areusch commented on pull request #29: Migrating Target Attributes to IRModule

2021-10-04 Thread GitBox
areusch commented on pull request #29: URL: https://github.com/apache/tvm-rfcs/pull/29#issuecomment-933753841 > [sorry, finger flub, sent too early] @areusch Got it. Yes, the 'configuration' needs to be fully under the user's control. > > I think I'm suggesting we need to collect the

[GitHub] [tvm] rkimball edited a comment on issue #9174: [Bug] [CPP RPC] custom_addr not properly quoted in JSON string

2021-10-04 Thread GitBox
rkimball edited a comment on issue #9174: URL: https://github.com/apache/tvm/issues/9174#issuecomment-933752701 You are correct. We need to add \\" around the `custom_addr` since it is a string. -- This is an automated message from the Apache Git Service. To respond to the message, pleas

[GitHub] [tvm] rkimball commented on issue #9174: [Bug] [CPP RPC] custom_addr not properly quoted in JSON string

2021-10-04 Thread GitBox
rkimball commented on issue #9174: URL: https://github.com/apache/tvm/issues/9174#issuecomment-933752701 You are correct. We need to add \" around the `custom_addr` since it is a string. -- This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [tvm] areusch commented on issue #9174: [Bug] [CPP RPC] custom_addr not properly quoted in JSON string

2021-10-04 Thread GitBox
areusch commented on issue #9174: URL: https://github.com/apache/tvm/issues/9174#issuecomment-933752603 thanks for reporting @lhez, @rkimball will take a look -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL ab

[GitHub] [tvm] areusch commented on pull request #9159: Address review comments on Arm(R) Ethos(TM)-U PR 3/6

2021-10-04 Thread GitBox
areusch commented on pull request #9159: URL: https://github.com/apache/tvm/pull/9159#issuecomment-933751071 cc @mbs-octoml @jroesch PTAL -- 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 spec

[GitHub] [tvm] areusch commented on issue #8953: [Bug] [microTVM] Truncated names of fused operators exceed CRT graph executor buffer lengths

2021-10-04 Thread GitBox
areusch commented on issue #8953: URL: https://github.com/apache/tvm/issues/8953#issuecomment-933749770 ah i think i see what happened--since we started including the mangling and post-TE-compiler refactor, we haven't updated `TVM_CRT_MAX_STRLEN_FUNCTION_NAME` to match the new world (where

[GitHub] [tvm-rfcs] areusch commented on a change in pull request #37: Introduce the Arm(R) Ethos(TM)-U Cascading Planner

2021-10-04 Thread GitBox
areusch commented on a change in pull request #37: URL: https://github.com/apache/tvm-rfcs/pull/37#discussion_r718677583 ## File path: rfcs/0037-arm-ethosu-cascading-planner.md ## @@ -41,51 +41,17 @@ Deciding on exactly which operators should be cascaded and with what striping

[GitHub] [tvm] mbs-octoml commented on a change in pull request #8788: [Relay] Remove DeviceMap from LowerTE

2021-10-04 Thread GitBox
mbs-octoml commented on a change in pull request #8788: URL: https://github.com/apache/tvm/pull/8788#discussion_r721521282 ## File path: src/relay/transforms/fuse_ops.cc ## @@ -1028,7 +1029,7 @@ Pass FuseOps(int fuse_opt_level) { auto max_fuse_depth = pc->GetConfig("re

[GitHub] [tvm] areusch commented on issue #9158: [CI Image] Update ci_i386 to 20210929-225912-da5b4fbfe

2021-10-04 Thread GitBox
areusch commented on issue #9158: URL: https://github.com/apache/tvm/issues/9158#issuecomment-933735204 cc @jtuyls -- 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 unsu

[GitHub] [tvm] areusch closed issue #9136: [CI] Update ci-cpu image to 20210921-225913-92ca78275

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

  1   2   >