[GitHub] [tvm] junrushao1994 commented on pull request #8778: [BUG] ToBasicBlockNormalForm immutability

2021-08-18 Thread GitBox
junrushao1994 commented on pull request #8778: URL: https://github.com/apache/tvm/pull/8778#issuecomment-901612903 Let's move fast and get it merged -- 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

[GitHub] [tvm] junrushao1994 commented on a change in pull request #8778: [BUG] ToBasicBlockNormalForm immutability

2021-08-18 Thread GitBox
junrushao1994 commented on a change in pull request #8778: URL: https://github.com/apache/tvm/pull/8778#discussion_r691782876 ## File path: src/relay/transforms/to_basic_block_normal_form.cc ## @@ -51,8 +51,11 @@ Expr ToBasicBlockNormalFormAux(const Expr& e) { IRModule

[GitHub] [tvm] ganler commented on a change in pull request #8778: [BUG] ToBasicBlockNormalForm immutability

2021-08-18 Thread GitBox
ganler commented on a change in pull request #8778: URL: https://github.com/apache/tvm/pull/8778#discussion_r691771235 ## File path: src/relay/transforms/to_basic_block_normal_form.cc ## @@ -51,8 +51,11 @@ Expr ToBasicBlockNormalFormAux(const Expr& e) { IRModule

[GitHub] [tvm] comaniac commented on pull request #8785: [FIX] Bug fix for batch_matmul parameters mismatch

2021-08-18 Thread GitBox
comaniac commented on pull request #8785: URL: https://github.com/apache/tvm/pull/8785#issuecomment-901576896 Thanks @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: [FIX] Bug fix for batch_matmul parameters mismatch (#8785)

2021-08-18 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 41879b2 [FIX] Bug fix for batch_matmul parameters

[GitHub] [tvm] comaniac merged pull request #8785: [FIX] Bug fix for batch_matmul parameters mismatch

2021-08-18 Thread GitBox
comaniac merged pull request #8785: URL: https://github.com/apache/tvm/pull/8785 -- 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] euntaik commented on a change in pull request #8780: [Frontend][TFLite] Implement fake quant

2021-08-18 Thread GitBox
euntaik commented on a change in pull request #8780: URL: https://github.com/apache/tvm/pull/8780#discussion_r691743077 ## File path: tests/python/frontend/tflite/test_forward.py ## @@ -2618,6 +2617,27 @@ def test_forward_select(): ) +def

[GitHub] [tvm-vta] aasorokiin opened a new pull request #32: VTA Chisel Wide memory interface.

2021-08-18 Thread GitBox
aasorokiin opened a new pull request #32: URL: https://github.com/apache/tvm-vta/pull/32 VTA modification for parametrizable AXI data size. Performance change: Numbers are based on test_benchmark_topi_conv2d.py limited to resnet-18.C2 Baseline 64bit data tsim run: 192M cycles.

[GitHub] [tvm] electriclilies commented on pull request #8775: Remove compile_engine.h for real this time.

2021-08-18 Thread GitBox
electriclilies commented on pull request #8775: URL: https://github.com/apache/tvm/pull/8775#issuecomment-901552531 LGTM, it's great to see that we've finally reached this milestone in the refactoring work! @comaniac Since we are trying to unify lowering around one API, I think that

[GitHub] [tvm] jcf94 commented on a change in pull request #8785: [FIX] Bug fix for batch_matmul parameters mismatch

2021-08-18 Thread GitBox
jcf94 commented on a change in pull request #8785: URL: https://github.com/apache/tvm/pull/8785#discussion_r691710645 ## File path: python/tvm/topi/cuda/batch_matmul.py ## @@ -258,11 +260,20 @@ def batch_matmul_int8(cfg, x, y, out_shape=None, out_dtype=None): out_dtype :

[GitHub] [tvm] junrushao1994 commented on pull request #8789: [CONTRIB] Add initializer to popenpool as well as test cases

2021-08-18 Thread GitBox
junrushao1994 commented on pull request #8789: URL: https://github.com/apache/tvm/pull/8789#issuecomment-901528736 @vinx13 this snippet does work, but i am not sure if it is the best way to do: ```python import importlib.util spec =

[GitHub] [tvm] vinx13 commented on pull request #8789: [CONTRIB] Add initializer to popenpool as well as test cases

2021-08-18 Thread GitBox
vinx13 commented on pull request #8789: URL: https://github.com/apache/tvm/pull/8789#issuecomment-901523306 > BTW, if we have a file containing user-registered functions, can we send this to the worker process so that the worker is aware of these registration? I suppose it is as simple as

[GitHub] [tvm] junrushao1994 commented on pull request #8789: [CONTRIB] Add initializer to popenpool as well as test cases

2021-08-18 Thread GitBox
junrushao1994 commented on pull request #8789: URL: https://github.com/apache/tvm/pull/8789#issuecomment-901518963 BTW, if we have a file containing user-registered functions, can we send this to the worker process so that the worker is aware of these registration? I suppose it is as

[GitHub] [tvm] junrushao1994 commented on a change in pull request #8789: [CONTRIB] Add initializer to popenpool as well as test cases

2021-08-18 Thread GitBox
junrushao1994 commented on a change in pull request #8789: URL: https://github.com/apache/tvm/pull/8789#discussion_r691691524 ## File path: python/tvm/contrib/popen_pool.py ## @@ -153,6 +160,11 @@ def _start(self): self._reader = os.fdopen(main_read, "rb")

[GitHub] [tvm] vinx13 commented on a change in pull request #8789: [CONTRIB] Add initializer to popenpool as well as test cases

2021-08-18 Thread GitBox
vinx13 commented on a change in pull request #8789: URL: https://github.com/apache/tvm/pull/8789#discussion_r691691114 ## File path: tests/python/contrib/test_popen_pool.py ## @@ -66,6 +66,19 @@ def test_popen_pool_executor(): assert val.value == idx +def

[GitHub] [tvm] vinx13 commented on a change in pull request #8789: [CONTRIB] Add initializer to popenpool as well as test cases

2021-08-18 Thread GitBox
vinx13 commented on a change in pull request #8789: URL: https://github.com/apache/tvm/pull/8789#discussion_r691688941 ## File path: python/tvm/testing/popen_pool.py ## @@ -0,0 +1,34 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor

[GitHub] [tvm] vinx13 commented on pull request #8789: [CONTRIB] Add initializer to popenpool as well as test cases

2021-08-18 Thread GitBox
vinx13 commented on pull request #8789: URL: https://github.com/apache/tvm/pull/8789#issuecomment-901515613 Thanks @shingjan , this will unblock refactor of using PopenPool in autotvm -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [tvm] vinx13 commented on a change in pull request #8789: [CONTRIB] Add initializer to popenpool as well as test cases

2021-08-18 Thread GitBox
vinx13 commented on a change in pull request #8789: URL: https://github.com/apache/tvm/pull/8789#discussion_r691688309 ## File path: python/tvm/contrib/popen_pool.py ## @@ -276,14 +288,19 @@ class PopenPoolExecutor: behavior of multiprocessing.pool(). """ -def

[GitHub] [tvm] junrushao1994 commented on pull request #8789: [CONTRIB] Add initializer to popenpool as well as test cases

2021-08-18 Thread GitBox
junrushao1994 commented on pull request #8789: URL: https://github.com/apache/tvm/pull/8789#issuecomment-901514283 Thanks @shingjan for the contribution! This work definitely makes Popen more complete, in terms of customizable initialization of workers :-) BTW, please document the

[GitHub] [tvm] shingjan opened a new pull request #8789: [CONTRIB] Add initializer to popenpool as well as test cases

2021-08-18 Thread GitBox
shingjan opened a new pull request #8789: URL: https://github.com/apache/tvm/pull/8789 This PR intends to add `initializer` as well as `initargs` to `popenpool` api. @tqchen @vinx13 @junrushao1994 -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [tvm] junrushao1994 commented on a change in pull request #8767: [TensorIR][M2a] Reorder

2021-08-18 Thread GitBox
junrushao1994 commented on a change in pull request #8767: URL: https://github.com/apache/tvm/pull/8767#discussion_r691686504 ## File path: src/tir/schedule/primitive/loop_transformation.cc ## @@ -385,6 +540,108 @@ StmtSRef Fuse(ScheduleState self, const Array& loop_srefs) {

[GitHub] [tvm] mbs-octoml opened a new pull request #8788: DRAFT [Relay] Remove DeviceMap from LowerTE

2021-08-18 Thread GitBox
mbs-octoml opened a new pull request #8788: URL: https://github.com/apache/tvm/pull/8788 Very rough sketch -- all I can say is it compiles... We continue the work sketched in

[tvm] branch main updated (7072743 -> 87674f9)

2021-08-18 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 7072743 Remove old AOT Executor code (#8758) add 87674f9 [microTVM] Project API Arduino support (#8708) No new

[GitHub] [tvm] areusch merged pull request #8708: [microTVM] Project API Arduino support

2021-08-18 Thread GitBox
areusch merged pull request #8708: URL: https://github.com/apache/tvm/pull/8708 -- 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] altanh commented on pull request #8778: [BUG] ToBasicBlockNormalForm immutability

2021-08-18 Thread GitBox
altanh commented on pull request #8778: URL: https://github.com/apache/tvm/pull/8778#issuecomment-901484084 agreed that `operator->() const` should not return a mutable pointer, wonder if this could be connected to various other similar problems we've had (e.g. AlterOpLayout). -- This

[GitHub] [tvm] areusch commented on pull request #8757: [CI] Modify Jenkinfile to always display junit report, fix for #8674

2021-08-18 Thread GitBox
areusch commented on pull request #8757: URL: https://github.com/apache/tvm/pull/8757#issuecomment-901471233 @mikepapadim I think this might still be busted. can you see if you can run it through a staging jenkins?

[tvm] branch main updated: Remove old AOT Executor code (#8758)

2021-08-18 Thread jroesch
This is an automated email from the ASF dual-hosted git repository. jroesch 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 7072743 Remove old AOT Executor code (#8758) 7072743

[GitHub] [tvm] jroesch merged pull request #8758: Remove old AOT Executor code

2021-08-18 Thread GitBox
jroesch merged pull request #8758: URL: https://github.com/apache/tvm/pull/8758 -- 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 #8758: Remove old AOT Executor code

2021-08-18 Thread GitBox
jroesch commented on pull request #8758: URL: https://github.com/apache/tvm/pull/8758#issuecomment-901456416 I am going to land this one to keep the PR Q moving, thanks guys! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [tvm] tkonolige commented on a change in pull request #8787: Fix typos

2021-08-18 Thread GitBox
tkonolige commented on a change in pull request #8787: URL: https://github.com/apache/tvm/pull/8787#discussion_r691620513 ## File path: include/tvm/node/reflection.h ## @@ -43,7 +43,7 @@ using runtime::ObjectPtr; using runtime::ObjectRef; /*! - * \brief Visitor class for

[GitHub] [tvm] jroesch edited a comment on pull request #8786: [CI] Rev ci-cpu to v0.76

2021-08-18 Thread GitBox
jroesch edited a comment on pull request #8786: URL: https://github.com/apache/tvm/pull/8786#issuecomment-901442412 @areusch ok cool, sorry for misunderstanding here @leandron. Can we add a check box or something as part of the template process until its fully automated so I/we can figure

[GitHub] [tvm] jroesch commented on pull request #8786: [CI] Rev ci-cpu to v0.76

2021-08-18 Thread GitBox
jroesch commented on pull request #8786: URL: https://github.com/apache/tvm/pull/8786#issuecomment-901442412 @areusch ok cool, sorry for misunderstanding here @leandron. Can we add a check box or whatever as part of the process until its fully automated so I can figure out if we need to

[GitHub] [tvm] areusch commented on pull request #8786: [CI] Rev ci-cpu to v0.76

2021-08-18 Thread GitBox
areusch commented on pull request #8786: URL: https://github.com/apache/tvm/pull/8786#issuecomment-901441461 ah--just clarifying for @jroesch. Jared, it's not generally true right now that the nightly build pipeline will run the built docker images through CI. @leandron is working

[tvm] branch main updated (5b2e504 -> 3f881ab)

2021-08-18 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 5b2e504 Restore License (#8779) add 3f881ab Expose FTVMInferCorrectLayout Python interface (#8755) No new

[GitHub] [tvm] comaniac commented on pull request #8755: Expose FTVMInferCorrectLayout Python interface

2021-08-18 Thread GitBox
comaniac commented on pull request #8755: URL: https://github.com/apache/tvm/pull/8755#issuecomment-901420771 Thanks @kueitang @junrushao1994 -- 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] comaniac merged pull request #8755: Expose FTVMInferCorrectLayout Python interface

2021-08-18 Thread GitBox
comaniac merged pull request #8755: URL: https://github.com/apache/tvm/pull/8755 -- 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] comaniac commented on pull request #8775: Remove compile_engine.h for real this time.

2021-08-18 Thread GitBox
comaniac commented on pull request #8775: URL: https://github.com/apache/tvm/pull/8775#issuecomment-901412908 I guess we cannot remove `compile_engine.py` but need to rename and refactor it, because some functions there, such as `lower_call` and `select_implementation` are still used by

[GitHub] [tvm] junrushao1994 commented on a change in pull request #8767: [TensorIR][M2a] Reorder

2021-08-18 Thread GitBox
junrushao1994 commented on a change in pull request #8767: URL: https://github.com/apache/tvm/pull/8767#discussion_r691580122 ## File path: src/tir/schedule/primitive/loop_transformation.cc ## @@ -253,6 +277,137 @@ class WrongFactorProductError : public ScheduleError { For

[GitHub] [tvm] junrushao1994 commented on a change in pull request #8767: [TensorIR][M2a] Reorder

2021-08-18 Thread GitBox
junrushao1994 commented on a change in pull request #8767: URL: https://github.com/apache/tvm/pull/8767#discussion_r691576602 ## File path: src/tir/schedule/primitive/loop_transformation.cc ## @@ -253,6 +277,137 @@ class WrongFactorProductError : public ScheduleError { For

[GitHub] [tvm] gromero opened a new pull request #8787: Fix typos

2021-08-18 Thread GitBox
gromero opened a new pull request #8787: URL: https://github.com/apache/tvm/pull/8787 Fix a couple of typos in comments about the IR/AST node reflection code and a typo in a comment about the main member of the TVMModule struct. Signed-off-by: Gustavo Romero -- This is an

[GitHub] [tvm] junrushao1994 commented on a change in pull request #8767: [TensorIR][M2a] Reorder

2021-08-18 Thread GitBox
junrushao1994 commented on a change in pull request #8767: URL: https://github.com/apache/tvm/pull/8767#discussion_r691552975 ## File path: src/tir/schedule/primitive/loop_transformation.cc ## @@ -155,7 +177,7 @@ class HasAnnotationOrThreadBindingError : public ScheduleError

[GitHub] [tvm] junrushao1994 commented on a change in pull request #8767: [TensorIR][M2a] Reorder

2021-08-18 Thread GitBox
junrushao1994 commented on a change in pull request #8767: URL: https://github.com/apache/tvm/pull/8767#discussion_r691550977 ## File path: src/tir/schedule/primitive.h ## @@ -64,6 +64,21 @@ TVM_DLL Array Split(ScheduleState self, const StmtSRef& loop_sref, * \return The

[GitHub] [tvm] junrushao1994 commented on a change in pull request #8767: [TensorIR][M2a] Reorder

2021-08-18 Thread GitBox
junrushao1994 commented on a change in pull request #8767: URL: https://github.com/apache/tvm/pull/8767#discussion_r691550055 ## File path: python/tvm/tir/schedule/schedule.py ## @@ -442,6 +442,64 @@ def after_split(a: ty.handle, b: ty.handle) -> None: # that there is

[GitHub] [tvm] junrushao1994 commented on a change in pull request #8767: [TensorIR][M2a] Reorder

2021-08-18 Thread GitBox
junrushao1994 commented on a change in pull request #8767: URL: https://github.com/apache/tvm/pull/8767#discussion_r691549866 ## File path: python/tvm/tir/schedule/schedule.py ## @@ -442,6 +442,64 @@ def after_split(a: ty.handle, b: ty.handle) -> None: # that there is

[GitHub] [tvm] junrushao1994 commented on a change in pull request #8767: [TensorIR][M2a] Reorder

2021-08-18 Thread GitBox
junrushao1994 commented on a change in pull request #8767: URL: https://github.com/apache/tvm/pull/8767#discussion_r691549659 ## File path: python/tvm/tir/schedule/schedule.py ## @@ -442,6 +442,64 @@ def after_split(a: ty.handle, b: ty.handle) -> None: # that there is

[GitHub] [tvm] junrushao1994 commented on a change in pull request #8767: [TensorIR][M2a] Reorder

2021-08-18 Thread GitBox
junrushao1994 commented on a change in pull request #8767: URL: https://github.com/apache/tvm/pull/8767#discussion_r691549171 ## File path: include/tvm/tir/schedule/schedule.h ## @@ -219,6 +219,19 @@ class ScheduleNode : public runtime::Object { * \return The new loops

[GitHub] [tvm] junrushao1994 commented on a change in pull request #8767: [TensorIR][M2a] Reorder

2021-08-18 Thread GitBox
junrushao1994 commented on a change in pull request #8767: URL: https://github.com/apache/tvm/pull/8767#discussion_r691548739 ## File path: include/tvm/tir/schedule/schedule.h ## @@ -219,6 +219,19 @@ class ScheduleNode : public runtime::Object { * \return The new loops

[GitHub] [tvm] junrushao1994 commented on a change in pull request #8767: [TensorIR][M2a] Reorder

2021-08-18 Thread GitBox
junrushao1994 commented on a change in pull request #8767: URL: https://github.com/apache/tvm/pull/8767#discussion_r691548093 ## File path: include/tvm/tir/schedule/schedule.h ## @@ -219,6 +219,19 @@ class ScheduleNode : public runtime::Object { * \return The new loops

[GitHub] [tvm] junrushao1994 commented on a change in pull request #8767: [TensorIR][M2a] Reorder

2021-08-18 Thread GitBox
junrushao1994 commented on a change in pull request #8767: URL: https://github.com/apache/tvm/pull/8767#discussion_r691547322 ## File path: include/tvm/tir/schedule/schedule.h ## @@ -219,6 +219,19 @@ class ScheduleNode : public runtime::Object { * \return The new loops

[GitHub] [tvm] areusch commented on pull request #8645: Add a PaddlePaddle Frontend

2021-08-18 Thread GitBox
areusch commented on pull request #8645: URL: https://github.com/apache/tvm/pull/8645#issuecomment-901374503 @junrushao1994 we now rebuild nightly to tlcpackstaging docker hub account. You can use those containers; however, you need to test them in CI first by pushing to

[GitHub] [tvm] junrushao1994 commented on pull request #8645: Add a PaddlePaddle Frontend

2021-08-18 Thread GitBox
junrushao1994 commented on pull request #8645: URL: https://github.com/apache/tvm/pull/8645#issuecomment-901368764 Hey @areusch @leandron, this PR depends on a re-build of the CI image to include paddle-paddle in our unittests (https://github.com/apache/tvm/pull/8742). Do you guys know

[GitHub] [tvm] junrushao1994 commented on pull request #8775: Remove compile_engine.h for real this time.

2021-08-18 Thread GitBox
junrushao1994 commented on pull request #8775: URL: https://github.com/apache/tvm/pull/8775#issuecomment-901367404 Thanks @comaniac for the list! It is definitely helpful for narrowing down the scope for discussion. Looks like we are pretty close to a solution to hot-swap the out-of-tree

[GitHub] [tvm] junrushao1994 commented on a change in pull request #8778: [BUG] ToBasicBlockNormalForm immutability

2021-08-18 Thread GitBox
junrushao1994 commented on a change in pull request #8778: URL: https://github.com/apache/tvm/pull/8778#discussion_r691522541 ## File path: src/relay/transforms/to_basic_block_normal_form.cc ## @@ -51,8 +51,11 @@ Expr ToBasicBlockNormalFormAux(const Expr& e) { IRModule

[GitHub] [tvm] junrushao1994 edited a comment on pull request #8778: [BUG] ToBasicBlockNormalForm immutability

2021-08-18 Thread GitBox
junrushao1994 edited a comment on pull request #8778: URL: https://github.com/apache/tvm/pull/8778#issuecomment-901351465 @Johnson9009 By design, we need to keep the immutability of AST instead of mutating it inplace, so I guess we should make it return a constant reference -- This is

[GitHub] [tvm] junrushao1994 commented on pull request #8778: [BUG] ToBasicBlockNormalForm immutability

2021-08-18 Thread GitBox
junrushao1994 commented on pull request #8778: URL: https://github.com/apache/tvm/pull/8778#issuecomment-901351465 @Johnson9009 By design, we need to keep the immutability of AST instead of mutating it inplace -- This is an automated message from the Apache Git Service. To respond to

[GitHub] [tvm] grant-arm commented on a change in pull request #8720: Sanitize names of input tensors in interface header

2021-08-18 Thread GitBox
grant-arm commented on a change in pull request #8720: URL: https://github.com/apache/tvm/pull/8720#discussion_r691516408 ## File path: python/tvm/micro/interface_api.py ## @@ -59,7 +60,8 @@ def generate_c_interface_header(module_name, inputs, outputs, output_path):

[GitHub] [tvm] leandron commented on pull request #8786: [CI] Rev ci-cpu to v0.76

2021-08-18 Thread GitBox
leandron commented on pull request #8786: URL: https://github.com/apache/tvm/pull/8786#issuecomment-901344200 This was tested in this previous job: https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/ci-docker-staging/146/pipeline Is the job wrong somehow? Asking so I can do

[GitHub] [tvm] comaniac commented on pull request #8775: Remove compile_engine.h for real this time.

2021-08-18 Thread GitBox
comaniac commented on pull request #8775: URL: https://github.com/apache/tvm/pull/8775#issuecomment-901324425 I've checked our use cases and here is my summary: - The use of APIs such as `_CompileEngineJIT ` can be replaced directly by `te_compiler.JIT`, so it should be safe to remove

[GitHub] [tvm-rfcs] AndrewZhaoLuo commented on pull request #6: [RFC] [Relay] Automatic Mixed Precision Pass

2021-08-18 Thread GitBox
AndrewZhaoLuo commented on pull request #6: URL: https://github.com/apache/tvm-rfcs/pull/6#issuecomment-901318475 @comaniac, I'll be talking about this at the TVM community meeting tomorrow so put off merging until after. -- This is an automated message from the Apache Git Service. To

[GitHub] [tvm-rfcs] AndrewZhaoLuo commented on a change in pull request #6: [RFC] [Relay] Automatic Mixed Precision Pass

2021-08-18 Thread GitBox
AndrewZhaoLuo commented on a change in pull request #6: URL: https://github.com/apache/tvm-rfcs/pull/6#discussion_r691486596 ## File path: rfcs/0001-AMP_pass.md ## @@ -0,0 +1,137 @@ +- Feature Name: Automatic Mixed Precision Pass +- Start Date: 2021-06-08 +- RFC PR: TODO +-

[GitHub] [tvm] jroesch commented on pull request #8786: [CI] Rev ci-cpu to v0.76

2021-08-18 Thread GitBox
jroesch commented on pull request #8786: URL: https://github.com/apache/tvm/pull/8786#issuecomment-901287493 After this goes green you should be gtg: https://github.com/apache/tvm/tree/ci-docker-staging -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [tvm] Mousius commented on pull request #8720: Sanitize names of input tensors in interface header

2021-08-18 Thread GitBox
Mousius commented on pull request #8720: URL: https://github.com/apache/tvm/pull/8720#issuecomment-901287191 I'd suggest we implement the test cases and the desired behaviour here and refactor it altogether in a later PR? It'll be a bit wonky either way and we can do a single refactor to

[tvm] 24/25: WIP

2021-08-18 Thread jroesch
This is an automated email from the ASF dual-hosted git repository. jroesch pushed a commit to branch ci-docker-staging in repository https://gitbox.apache.org/repos/asf/tvm.git commit 9ebc663f18630e24fea62b5bac27b1fe8a4896a9 Author: Jared Roesch AuthorDate: Wed Mar 31 22:51:06 2021 -0700

[tvm] 22/25: Tweak CI

2021-08-18 Thread jroesch
This is an automated email from the ASF dual-hosted git repository. jroesch pushed a commit to branch ci-docker-staging in repository https://gitbox.apache.org/repos/asf/tvm.git commit 8f8333f9402d4e6224a165dfc3643ae32e68caa7 Author: Jared Roesch AuthorDate: Tue Mar 30 13:53:21 2021 -0700

[tvm] 20/25: Clean up build

2021-08-18 Thread jroesch
This is an automated email from the ASF dual-hosted git repository. jroesch pushed a commit to branch ci-docker-staging in repository https://gitbox.apache.org/repos/asf/tvm.git commit 6b8c29582c1b0a86d92803748bfcbf3658f06f94 Author: Jared Roesch AuthorDate: Fri Mar 26 15:13:32 2021 -0700

[tvm] 25/25: Format

2021-08-18 Thread jroesch
This is an automated email from the ASF dual-hosted git repository. jroesch pushed a commit to branch ci-docker-staging in repository https://gitbox.apache.org/repos/asf/tvm.git commit adf12e40b50701fb2dbc0464617b668cda9ad2b0 Author: Jared Roesch AuthorDate: Wed Mar 31 23:28:22 2021 -0700

[tvm] 21/25: Turn docs back on

2021-08-18 Thread jroesch
This is an automated email from the ASF dual-hosted git repository. jroesch pushed a commit to branch ci-docker-staging in repository https://gitbox.apache.org/repos/asf/tvm.git commit 74932d023033f85431bf7675a8614b0955226052 Author: Jared Roesch AuthorDate: Fri Mar 26 15:14:11 2021 -0700

[tvm] 19/25: Format

2021-08-18 Thread jroesch
This is an automated email from the ASF dual-hosted git repository. jroesch pushed a commit to branch ci-docker-staging in repository https://gitbox.apache.org/repos/asf/tvm.git commit 34806acef5cb991c72cedfd9bd9e7bb14c95af62 Author: Jared Roesch AuthorDate: Fri Mar 26 14:41:25 2021 -0700

[tvm] 17/25: Add header

2021-08-18 Thread jroesch
This is an automated email from the ASF dual-hosted git repository. jroesch pushed a commit to branch ci-docker-staging in repository https://gitbox.apache.org/repos/asf/tvm.git commit b8613d61988b10d7a82736c2ccbaa28cf6e243b7 Author: Jared Roesch AuthorDate: Fri Mar 26 14:36:42 2021 -0700

[tvm] 15/25: Conform to Apache branding guidelines

2021-08-18 Thread jroesch
This is an automated email from the ASF dual-hosted git repository. jroesch pushed a commit to branch ci-docker-staging in repository https://gitbox.apache.org/repos/asf/tvm.git commit dd7cc316367ac7ecd2eb5b1d1b344e798ea1cdf0 Author: Jared Roesch AuthorDate: Tue Feb 23 13:56:23 2021 -0800

[tvm] 23/25: Bump the Jenkinsfile

2021-08-18 Thread jroesch
This is an automated email from the ASF dual-hosted git repository. jroesch pushed a commit to branch ci-docker-staging in repository https://gitbox.apache.org/repos/asf/tvm.git commit f8522e9b74485453f3e262177b5a07d1d075a806 Author: Jared Roesch AuthorDate: Wed Mar 31 14:13:43 2021 -0700

[tvm] 16/25: Fix caps

2021-08-18 Thread jroesch
This is an automated email from the ASF dual-hosted git repository. jroesch pushed a commit to branch ci-docker-staging in repository https://gitbox.apache.org/repos/asf/tvm.git commit 9dee4995dbfd69f5a589f9249e8d109f949d1cb1 Author: Jared Roesch AuthorDate: Tue Feb 23 14:08:13 2021 -0800

[tvm] 12/25: Tweak versions for publishing

2021-08-18 Thread jroesch
This is an automated email from the ASF dual-hosted git repository. jroesch pushed a commit to branch ci-docker-staging in repository https://gitbox.apache.org/repos/asf/tvm.git commit 0b593de286daf4f10361af31f601d9c62975d6c2 Author: Jared Roesch AuthorDate: Mon Feb 22 23:34:50 2021 -0800

[tvm] 09/25: Move generated code to OUT_DIR

2021-08-18 Thread jroesch
This is an automated email from the ASF dual-hosted git repository. jroesch pushed a commit to branch ci-docker-staging in repository https://gitbox.apache.org/repos/asf/tvm.git commit 09ff91475706a826636e7f04beb68447fd1a67e8 Author: Jared Roesch AuthorDate: Mon Feb 22 23:04:07 2021 -0800

[tvm] 18/25: Remove warning

2021-08-18 Thread jroesch
This is an automated email from the ASF dual-hosted git repository. jroesch pushed a commit to branch ci-docker-staging in repository https://gitbox.apache.org/repos/asf/tvm.git commit 6d639ba9dabb73c4f89eff43c3548ee6ed75a598 Author: Jared Roesch AuthorDate: Fri Mar 26 14:40:27 2021 -0700

[tvm] 13/25: Tweak versions for publishing

2021-08-18 Thread jroesch
This is an automated email from the ASF dual-hosted git repository. jroesch pushed a commit to branch ci-docker-staging in repository https://gitbox.apache.org/repos/asf/tvm.git commit c281b2d40e796f8b52ccd21b206d35a0e402e9e4 Author: Jared Roesch AuthorDate: Tue Feb 23 10:45:10 2021 -0800

[tvm] 14/25: Add README for tvm-graph-rt

2021-08-18 Thread jroesch
This is an automated email from the ASF dual-hosted git repository. jroesch pushed a commit to branch ci-docker-staging in repository https://gitbox.apache.org/repos/asf/tvm.git commit b540eefd0dabeded73d4585b744b73581131a80b Author: Jared Roesch AuthorDate: Tue Feb 23 10:46:38 2021 -0800

[tvm] 10/25: Fix path

2021-08-18 Thread jroesch
This is an automated email from the ASF dual-hosted git repository. jroesch pushed a commit to branch ci-docker-staging in repository https://gitbox.apache.org/repos/asf/tvm.git commit 309eff02e46e23212083f4aa40424a980c1e701e Author: Jared Roesch AuthorDate: Mon Feb 22 23:08:49 2021 -0800

[tvm] 08/25: Add README

2021-08-18 Thread jroesch
This is an automated email from the ASF dual-hosted git repository. jroesch pushed a commit to branch ci-docker-staging in repository https://gitbox.apache.org/repos/asf/tvm.git commit a8391af4bff2e25a9f4494b21544dc62c877265b Author: Jared Roesch AuthorDate: Mon Feb 22 22:58:33 2021 -0800

[tvm] 11/25: Add descp for tvm-sys

2021-08-18 Thread jroesch
This is an automated email from the ASF dual-hosted git repository. jroesch pushed a commit to branch ci-docker-staging in repository https://gitbox.apache.org/repos/asf/tvm.git commit 8e373c848f3880fa5ce51c7981aa11b9d38d5828 Author: Jared Roesch AuthorDate: Mon Feb 22 23:10:39 2021 -0800

[tvm] 04/25: Tweak build.rs to use release version of tvm-build

2021-08-18 Thread jroesch
This is an automated email from the ASF dual-hosted git repository. jroesch pushed a commit to branch ci-docker-staging in repository https://gitbox.apache.org/repos/asf/tvm.git commit 03b7019ba0304b634d3cf4d0893cc808e7729f21 Author: Jared Roesch AuthorDate: Mon Feb 22 21:34:29 2021 -0800

[tvm] 03/25: Update build.rs to use new tvm-build version

2021-08-18 Thread jroesch
This is an automated email from the ASF dual-hosted git repository. jroesch pushed a commit to branch ci-docker-staging in repository https://gitbox.apache.org/repos/asf/tvm.git commit 1939a4d5eb0965b9f2c11e042dc2dcb00797f1f5 Author: Jared Roesch AuthorDate: Sat Feb 20 01:09:58 2021 -0800

[tvm] 07/25: Fix Cargo verisions for pre-release

2021-08-18 Thread jroesch
This is an automated email from the ASF dual-hosted git repository. jroesch pushed a commit to branch ci-docker-staging in repository https://gitbox.apache.org/repos/asf/tvm.git commit cabc9a7726f1321747353e73020792fde63df535 Author: Jared Roesch AuthorDate: Mon Feb 22 22:56:56 2021 -0800

[tvm] 05/25: Add docs

2021-08-18 Thread jroesch
This is an automated email from the ASF dual-hosted git repository. jroesch pushed a commit to branch ci-docker-staging in repository https://gitbox.apache.org/repos/asf/tvm.git commit ab1bc811230f6b4df84ce353d17be7611db4ddde Author: Jared Roesch AuthorDate: Mon Feb 22 22:49:23 2021 -0800

[tvm] 06/25: Add Readme for tvm-sys crate.

2021-08-18 Thread jroesch
This is an automated email from the ASF dual-hosted git repository. jroesch pushed a commit to branch ci-docker-staging in repository https://gitbox.apache.org/repos/asf/tvm.git commit 954abe0e669c6dba7f7ed05863411ae375e7e1b4 Author: Jared Roesch AuthorDate: Mon Feb 22 22:49:40 2021 -0800

[tvm] 01/25: Rewrite the Rust Module API and change some imports causing crashes.

2021-08-18 Thread jroesch
This is an automated email from the ASF dual-hosted git repository. jroesch pushed a commit to branch ci-docker-staging in repository https://gitbox.apache.org/repos/asf/tvm.git commit 69675d46ea9161234cbfaeb9c8c65897a0eb99b0 Author: Jared Roesch AuthorDate: Wed Feb 17 21:01:22 2021 -0800

[tvm] 02/25: Fixes for version bump

2021-08-18 Thread jroesch
This is an automated email from the ASF dual-hosted git repository. jroesch pushed a commit to branch ci-docker-staging in repository https://gitbox.apache.org/repos/asf/tvm.git commit 5651bd62ad698587ee4eeb8c157192d90a0d3cc0 Author: Jared Roesch AuthorDate: Thu Feb 18 22:49:57 2021 -0800

[GitHub] [tvm] jroesch commented on pull request #8786: [CI] Rev ci-cpu to v0.76

2021-08-18 Thread GitBox
jroesch commented on pull request #8786: URL: https://github.com/apache/tvm/pull/8786#issuecomment-901282649 @leandron this hasn't actually tested that the image works on master, we need to wait on a `ci-staging` push before merging correct? -- This is an automated message from the

[GitHub] [tvm] jroesch commented on issue #8782: [CI] Rev ci-cpu image to include latest PRs

2021-08-18 Thread GitBox
jroesch commented on issue #8782: URL: https://github.com/apache/tvm/issues/8782#issuecomment-901282048 @leandron its currently uploading to dockerhub -- 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 commented on a change in pull request #8768: [TIR] Fix buffer scope in structural equal

2021-08-18 Thread GitBox
tqchen commented on a change in pull request #8768: URL: https://github.com/apache/tvm/pull/8768#discussion_r691438829 ## File path: include/tvm/ir/type.h ## @@ -164,10 +164,17 @@ class PointerTypeNode : public TypeNode { } bool SEqualReduce(const PointerTypeNode*

[GitHub] [tvm] leandron opened a new pull request #8786: [CI] Rev ci-cpu to v0.76

2021-08-18 Thread GitBox
leandron opened a new pull request #8786: URL: https://github.com/apache/tvm/pull/8786 * This includes changes up to commit 1a95f9bd0. * Tracking issue is https://github.com/apache/tvm/issues/8782 **Important: there is some ongoing pre-work being done by @jroesch before this can

[GitHub] [tvm] comaniac commented on a change in pull request #8785: [FIX] Bug fix for batch_matmul parameters mismatch

2021-08-18 Thread GitBox
comaniac commented on a change in pull request #8785: URL: https://github.com/apache/tvm/pull/8785#discussion_r691420715 ## File path: python/tvm/topi/cuda/batch_matmul.py ## @@ -258,11 +260,20 @@ def batch_matmul_int8(cfg, x, y, out_shape=None, out_dtype=None):

[GitHub] [tvm] denise-k commented on pull request #8784: [Docker][Vulkan] Allow Vulkan GPU access in docker container.

2021-08-18 Thread GitBox
denise-k commented on pull request #8784: URL: https://github.com/apache/tvm/pull/8784#issuecomment-901266341 @Lunderberg thanks for opening this PR. In my opinion, it's an overall net positive to have the Vulkan tests in CI. However, until we have good data on how much run time Vulkan

[GitHub] [tvm] junrushao1994 commented on pull request #8704: [Relay testing] densenet implementation fix

2021-08-18 Thread GitBox
junrushao1994 commented on pull request #8704: URL: https://github.com/apache/tvm/pull/8704#issuecomment-900416980 -- 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

[GitHub] [tvm] tkonolige commented on pull request #8770: [contrib] Fix "custom_path" already exist bug

2021-08-18 Thread GitBox
tkonolige commented on pull request #8770: URL: https://github.com/apache/tvm/pull/8770#issuecomment-900478536 I believe the correct way to do this is [`os.makedirs`](https://docs.python.org/3/library/os.html#os.makedirs) with `exist_ok=True`. -- This is an automated message from the

[GitHub] [tvm] leandron commented on issue #8783: Error emitting Optimized C Code

2021-08-18 Thread GitBox
leandron commented on issue #8783: URL: https://github.com/apache/tvm/issues/8783#issuecomment-901008313 Hi @bhahn221. I had a look into this issue and it seems like some similar issues we've seen in past with `tvmc compile --target=c`. Based on the error message, it looks like we

[GitHub] [tvm-rfcs] manupa-arm commented on pull request #22: [RFC][TIR] TIR Non-scalar Constants

2021-08-18 Thread GitBox
manupa-arm commented on pull request #22: URL: https://github.com/apache/tvm-rfcs/pull/22#issuecomment-900469754 cc : @tqchen @junrushao1994 @jroesch @areusch -- 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] mikepapadim commented on a change in pull request #8775: Remove compile_engine.h for real this time.

2021-08-18 Thread GitBox
mikepapadim commented on a change in pull request #8775: URL: https://github.com/apache/tvm/pull/8775#discussion_r691167727 ## File path: src/relay/backend/compile_engine.cc ## @@ -1,338 +0,0 @@ -/* - * Licensed to the Apache Software Foundation (ASF) under one - * or more

[GitHub] [tvm] huanmei9 edited a comment on pull request #8770: [contrib] Fix "custom_path" already exist bug

2021-08-18 Thread GitBox
huanmei9 edited a comment on pull request #8770: URL: https://github.com/apache/tvm/pull/8770#issuecomment-900758591 Hi @tkonolige @junrushao1994 . Thanks for your suggestions, I have changed the code according to your suggestions. -- This is an automated message from the Apache Git

[GitHub] [tvm] Mousius commented on issue #8783: Error emitting Optimized C Code

2021-08-18 Thread GitBox
Mousius commented on issue #8783: URL: https://github.com/apache/tvm/issues/8783#issuecomment-901006833 -- 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,

  1   2   3   >