[GitHub] [tvm] zxybazh commented on issue #13639: [Bug] cudaFree() error when use meta-schedule

2023-01-06 Thread GitBox
zxybazh commented on issue #13639: URL: https://github.com/apache/tvm/issues/13639#issuecomment-1374403257 Yes I can see the same warning `CUDA driver initialization failed, you might not have a CUDA gpu.` I can also get `torch.cuda.is_available()` returning `True` for me. When using your s

[GitHub] [tvm] vinx13 commented on issue #13639: [Bug] cudaFree() error when use meta-schedule

2023-01-06 Thread GitBox
vinx13 commented on issue #13639: URL: https://github.com/apache/tvm/issues/13639#issuecomment-1374399720 @masahi i actually got the same warning but it runs well. I’m using PyTorch built from source -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [tvm] junrushao commented on issue #13639: [Bug] cudaFree() error when use meta-schedule

2023-01-06 Thread GitBox
junrushao commented on issue #13639: URL: https://github.com/apache/tvm/issues/13639#issuecomment-1374393643 That could be very likely the cause, and that's why we usually carefully isolate PyTorch in a separate process to avoid such problems in MetaSchedule experiments. -- This is an au

[tvm] branch main updated: [Tests] Replace pytest.main with tvm.testing.main (#13717)

2023-01-06 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 bd37515cfb [Tests] Replace pytest.main with tvm.test

[GitHub] [tvm] junrushao merged pull request #13717: [Tests] Replace pytest.main with tvm.testing.main

2023-01-06 Thread GitBox
junrushao merged PR #13717: URL: https://github.com/apache/tvm/pull/13717 -- 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.apache

[tvm] branch main updated: [TVMScript] Linter-friendly function definitions (#13713)

2023-01-06 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 875296c762 [TVMScript] Linter-friendly function defi

[GitHub] [tvm] junrushao merged pull request #13713: [TVMScript] Linter-friendly function definitions

2023-01-06 Thread GitBox
junrushao merged PR #13713: URL: https://github.com/apache/tvm/pull/13713 -- 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.apache

[GitHub] [tvm] junrushao commented on a diff in pull request #13593: [TVMScript] Refactor IRDocsifier

2023-01-06 Thread GitBox
junrushao commented on code in PR #13593: URL: https://github.com/apache/tvm/pull/13593#discussion_r1063957932 ## include/tvm/script/printer/printer.h: ## @@ -0,0 +1,82 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more contributor license agreemen

[GitHub] [tvm] Hzfengsy commented on a diff in pull request #13593: [TVMScript] Refactor IRDocsifier

2023-01-06 Thread GitBox
Hzfengsy commented on code in PR #13593: URL: https://github.com/apache/tvm/pull/13593#discussion_r1063957422 ## include/tvm/script/printer/printer.h: ## @@ -0,0 +1,82 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more contributor license agreement

[GitHub] [tvm] junrushao commented on pull request #13713: [TVMScript] Linter-friendly function definitions

2023-01-06 Thread GitBox
junrushao commented on PR #13713: URL: https://github.com/apache/tvm/pull/13713#issuecomment-1374385302 Thanks for the PR! I will verify it myself before approving :-) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use th

[GitHub] [tvm] junrushao commented on a diff in pull request #13593: [TVMScript] Refactor IRDocsifier

2023-01-06 Thread GitBox
junrushao commented on code in PR #13593: URL: https://github.com/apache/tvm/pull/13593#discussion_r1063956609 ## src/script/printer/doc_printer/python_doc_printer.cc: ## @@ -294,7 +295,11 @@ void PythonDocPrinter::PrintTypedDoc(const LiteralDoc& doc) { } else if (const auto

[GitHub] [tvm] junrushao commented on a diff in pull request #13593: [TVMScript] Refactor IRDocsifier

2023-01-06 Thread GitBox
junrushao commented on code in PR #13593: URL: https://github.com/apache/tvm/pull/13593#discussion_r1063956581 ## include/tvm/script/printer/printer.h: ## @@ -0,0 +1,82 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more contributor license agreemen

[GitHub] [tvm] junrushao commented on a diff in pull request #13593: [TVMScript] Refactor IRDocsifier

2023-01-06 Thread GitBox
junrushao commented on code in PR #13593: URL: https://github.com/apache/tvm/pull/13593#discussion_r1063956422 ## tests/scripts/task_mypy.sh: ## @@ -32,9 +32,6 @@ mypy --check-untyped-defs python/tvm/tir/analysis/ echo "Checking MyPy Type defs in the transform package." mypy

[tvm] branch main updated: [Fix,AutoScheduler] Handle 0-dim buffers in featurization (#13718)

2023-01-06 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 843310b03c [Fix,AutoScheduler] Handle 0-dim buffers in

[GitHub] [tvm] masahi merged pull request #13718: [Fix,AutoScheduler] Handle 0-dim buffers in featurization

2023-01-06 Thread GitBox
masahi merged PR #13718: URL: https://github.com/apache/tvm/pull/13718 -- 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.apache.or

[GitHub] [tvm] masahi commented on issue #13639: [Bug] cudaFree() error when use meta-schedule

2023-01-06 Thread GitBox
masahi commented on issue #13639: URL: https://github.com/apache/tvm/issues/13639#issuecomment-1374381932 It seems the error happens before or at the very start of `run` - I added a debug print to see at which kernel this error is raised, but nothing is logged before the error is emitted.

[tvm] branch nightly updated (21d7968b61 -> 088bc118c7)

2023-01-06 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch nightly in repository https://gitbox.apache.org/repos/asf/tvm.git from 21d7968b61 [microTVM] Fix MacOS build with USE_MICRO=ON (#13711) add 123f1f5e2c [tir] Add line level debug info

[tvm] branch main updated: [TIR] Fix dtype mismatch error due to LetStmt (#13710)

2023-01-06 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 088bc118c7 [TIR] Fix dtype mismatch error due to LetStm

[GitHub] [tvm] masahi merged pull request #13710: [TIR] Fix dtype mismatch error due to LetStmt

2023-01-06 Thread GitBox
masahi merged PR #13710: URL: https://github.com/apache/tvm/pull/13710 -- 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.apache.or

[GitHub] [tvm] Hzfengsy commented on a diff in pull request #13593: [TVMScript] Refactor IRDocsifier

2023-01-06 Thread GitBox
Hzfengsy commented on code in PR #13593: URL: https://github.com/apache/tvm/pull/13593#discussion_r1063948765 ## include/tvm/script/printer/printer.h: ## @@ -0,0 +1,82 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more contributor license agreement

[GitHub] [tvm] Lunderberg commented on pull request #13708: [Arith] Simplify to positive numerators in floordiv/floormod

2023-01-06 Thread GitBox
Lunderberg commented on PR #13708: URL: https://github.com/apache/tvm/pull/13708#issuecomment-1374370669 The latest change to the simplification caused more breakages than I think it is worth. Closing this PR in favor of https://github.com/apache/tvm/pull/13724, which has a more limited sc

[GitHub] [tvm] Lunderberg closed pull request #13708: [Arith] Simplify to positive numerators in floordiv/floormod

2023-01-06 Thread GitBox
Lunderberg closed pull request #13708: [Arith] Simplify to positive numerators in floordiv/floormod URL: https://github.com/apache/tvm/pull/13708 -- 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] tvm-bot commented on pull request #13724: [Arith] Use ConstIntBound to remove negative numerator when lowering

2023-01-06 Thread GitBox
tvm-bot commented on PR #13724: URL: https://github.com/apache/tvm/pull/13724#issuecomment-1374370525 Thanks for contributing to TVM! Please refer to the contributing guidelines https://tvm.apache.org/docs/contribute/ for useful information and tips. Please request code reviews from

[GitHub] [tvm] Lunderberg opened a new pull request, #13724: [Arith] Use ConstIntBound to remove negative numerator when lowering

2023-01-06 Thread GitBox
Lunderberg opened a new pull request, #13724: URL: https://github.com/apache/tvm/pull/13724 Negative numerators to modulo/remainder operations are not supported by the Vulkan API. While the SPIR-V instructions [`OpSRem`](https://registry.khronos.org/SPIR-V/specs/unified1/SPIRV.html#OpSR

[GitHub] [tvm] junrushao commented on issue #13639: [Bug] cudaFree() error when use meta-schedule

2023-01-06 Thread GitBox
junrushao commented on issue #13639: URL: https://github.com/apache/tvm/issues/13639#issuecomment-1374344623 Can we try: ```diff - module.run() - return module.get_output(0).numpy() + module.run() + result = module.get_output(0).numpy() + del module + return result

[GitHub] [tvm] mehrdadh opened a new pull request, #13723: [microTVM][Zephyr]Fix flash command for nrfjprog

2023-01-06 Thread GitBox
mehrdadh opened a new pull request, #13723: URL: https://github.com/apache/tvm/pull/13723 This PR adds serial number to flash command (nrfjprog) to fix cases where multiple devices are available. cc @gromero -- This is an automated message from the Apache Git Service. To respond t

[GitHub] [tvm] tvm-bot commented on pull request #13723: [microTVM][Zephyr]Fix flash command for nrfjprog

2023-01-06 Thread GitBox
tvm-bot commented on PR #13723: URL: https://github.com/apache/tvm/pull/13723#issuecomment-1374342019 Thanks for contributing to TVM! Please refer to the contributing guidelines https://tvm.apache.org/docs/contribute/ for useful information and tips. Please request code reviews from

[tvm] 01/01: Fix NRF find serial

2023-01-06 Thread mehrdadh
This is an automated email from the ASF dual-hosted git repository. mehrdadh pushed a commit to branch micro/fix_nrf_flash in repository https://gitbox.apache.org/repos/asf/tvm.git commit 1881fd55c10784e5e47ef4dfa1507e456a258386 Author: Mehrdad Hessar AuthorDate: Fri Jan 6 16:42:51 2023 -0800

[tvm] branch micro/fix_nrf_flash updated (37c11f8cfe -> 1881fd55c1)

2023-01-06 Thread mehrdadh
This is an automated email from the ASF dual-hosted git repository. mehrdadh pushed a change to branch micro/fix_nrf_flash in repository https://gitbox.apache.org/repos/asf/tvm.git discard 37c11f8cfe Fix NRF find serial new 1881fd55c1 Fix NRF find serial This update added new revisions af

[tvm] branch main updated: [docs] Add "Open with Colab" button to documentation (#13627)

2023-01-06 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.git The following commit(s) were added to refs/heads/main by this push: new 30abbe9832 [docs] Add "Open with Colab" button to docu

[GitHub] [tvm] areusch merged pull request #13627: [docs] Add "Open with Colab" button to documentation

2023-01-06 Thread GitBox
areusch merged PR #13627: URL: https://github.com/apache/tvm/pull/13627 -- 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.apache.o

[GitHub] [tvm] areusch commented on a diff in pull request #13627: [docs] Add "Open with Colab" button to documentation

2023-01-06 Thread GitBox
areusch commented on code in PR #13627: URL: https://github.com/apache/tvm/pull/13627#discussion_r1060859591 ## gallery/how_to/compile_models/from_coreml.py: ## @@ -23,13 +23,12 @@ This article is an introductory tutorial to deploy CoreML models with Relay. -For us to begin

[tvm] branch micro/fix_nrf_flash updated (4370c4c0df -> 37c11f8cfe)

2023-01-06 Thread mehrdadh
This is an automated email from the ASF dual-hosted git repository. mehrdadh pushed a change to branch micro/fix_nrf_flash in repository https://gitbox.apache.org/repos/asf/tvm.git discard 4370c4c0df Fix omit 16ebf61c1e cleanup omit a6bc1719dd Update nrfjprog script and LLVM script

[tvm] 01/01: Fix NRF find serial

2023-01-06 Thread mehrdadh
This is an automated email from the ASF dual-hosted git repository. mehrdadh pushed a commit to branch micro/fix_nrf_flash in repository https://gitbox.apache.org/repos/asf/tvm.git commit 37c11f8cfe5c35c3578cbda680628b83398d3880 Author: Mehrdad Hessar AuthorDate: Fri Jan 6 16:42:51 2023 -0800

[GitHub] [tvm] tvm-bot commented on pull request #13722: [CI][Docker][Cortex-M]Update ci_cortexm to Ubuntu 20.04 and update nrfjprog version to 10.18.1

2023-01-06 Thread GitBox
tvm-bot commented on PR #13722: URL: https://github.com/apache/tvm/pull/13722#issuecomment-1374333740 Thanks for contributing to TVM! Please refer to the contributing guidelines https://tvm.apache.org/docs/contribute/ for useful information and tips. Please request code reviews from

[GitHub] [tvm] mehrdadh opened a new pull request, #13722: [CI][Docker][Cortex-M]Update ci_cortexm to Ubuntu 20.04 and update nrfjprog version to 10.18.1

2023-01-06 Thread GitBox
mehrdadh opened a new pull request, #13722: URL: https://github.com/apache/tvm/pull/13722 This PR updates nrfjprog to latest (10.18.1) to fix issues related to running microTVM tests inside docker. The main issue was that with older version commands like `nrfjprog --com` was not working eve

[tvm] branch ci/update_cortexm_ubuntu20 updated (a6bc1719dd -> 16ebf61c1e)

2023-01-06 Thread mehrdadh
This is an automated email from the ASF dual-hosted git repository. mehrdadh pushed a change to branch ci/update_cortexm_ubuntu20 in repository https://gitbox.apache.org/repos/asf/tvm.git from a6bc1719dd Update nrfjprog script and LLVM script add 16ebf61c1e cleanup No new revisions wer

[GitHub] [tvm] nverke closed pull request #13446: [MetaSchedule] Add MultiLevelTilingHexagon to create simple schedules…

2023-01-06 Thread GitBox
nverke closed pull request #13446: [MetaSchedule] Add MultiLevelTilingHexagon to create simple schedules… URL: https://github.com/apache/tvm/pull/13446 -- 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 t

[GitHub] [tvm] nverke opened a new pull request, #13721: [MetaScheduler][Hexagon] Add MultiLevelTilingHexagon to schedule asyn…

2023-01-06 Thread GitBox
nverke opened a new pull request, #13721: URL: https://github.com/apache/tvm/pull/13721 …c pipelines that utilize DMA This will create schedules that utilize async dma pipelines on hexagon. Currently this has some limitations due to the need to have dma lowering be contiguous memory

[GitHub] [tvm] tvm-bot commented on pull request #13721: [MetaScheduler][Hexagon] Add MultiLevelTilingHexagon to schedule asyn…

2023-01-06 Thread GitBox
tvm-bot commented on PR #13721: URL: https://github.com/apache/tvm/pull/13721#issuecomment-1374326585 Thanks for contributing to TVM! Please refer to the contributing guidelines https://tvm.apache.org/docs/contribute/ for useful information and tips. Please request code reviews from

[GitHub] [tvm] tvm-bot commented on pull request #13720: Add DisallowAsyncStridedMemCopy post processor to rem

2023-01-06 Thread GitBox
tvm-bot commented on PR #13720: URL: https://github.com/apache/tvm/pull/13720#issuecomment-1374318961 Thanks for contributing to TVM! Please refer to the contributing guidelines https://tvm.apache.org/docs/contribute/ for useful information and tips. Please request code reviews from

[GitHub] [tvm] nverke opened a new pull request, #13720: Add DisallowAsyncStridedMemCopy post processor to rem

2023-01-06 Thread GitBox
nverke opened a new pull request, #13720: URL: https://github.com/apache/tvm/pull/13720 …ove schedules that use async strided mem copies. This is used to remove all of the schedules that would be created by MS that would violate the contiguous requirement of async DMA copies on hexago

[tvm] branch main updated (123f1f5e2c -> 52d8485e48)

2023-01-06 Thread mehrdadh
This is an automated email from the ASF dual-hosted git repository. mehrdadh pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/tvm.git from 123f1f5e2c [tir] Add line level debug info (#13012) add 52d8485e48 [microTVM][Zephyr]Add project files for mlperftin

[GitHub] [tvm] mehrdadh merged pull request #13690: [microTVM][Zephyr]Add project files for mlperftiny submission

2023-01-06 Thread GitBox
mehrdadh merged PR #13690: URL: https://github.com/apache/tvm/pull/13690 -- 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.apache.

[tvm] 02/02: Fix

2023-01-06 Thread mehrdadh
This is an automated email from the ASF dual-hosted git repository. mehrdadh pushed a commit to branch micro/fix_nrf_flash in repository https://gitbox.apache.org/repos/asf/tvm.git commit 4370c4c0df205b4177bfedce60a0fce75eef95be Author: Mehrdad Hessar AuthorDate: Fri Jan 6 16:42:51 2023 -0800

[tvm] 01/02: cleanup

2023-01-06 Thread mehrdadh
This is an automated email from the ASF dual-hosted git repository. mehrdadh pushed a commit to branch micro/fix_nrf_flash in repository https://gitbox.apache.org/repos/asf/tvm.git commit 16ebf61c1e619861292c6d842f8b41a34d726c98 Author: Mehrdad Hessar AuthorDate: Fri Jan 6 15:06:20 2023 -0800

[tvm] branch micro/fix_nrf_flash created (now 4370c4c0df)

2023-01-06 Thread mehrdadh
This is an automated email from the ASF dual-hosted git repository. mehrdadh pushed a change to branch micro/fix_nrf_flash in repository https://gitbox.apache.org/repos/asf/tvm.git at 4370c4c0df Fix This branch includes the following new commits: new 16ebf61c1e cleanup new 4370

[GitHub] [tvm] masahi commented on a diff in pull request #13450: [COLLAGE] Add more customization to support more targets

2023-01-06 Thread GitBox
masahi commented on code in PR #13450: URL: https://github.com/apache/tvm/pull/13450#discussion_r1063893354 ## src/relay/collage/custom_cost_estimator.cc: ## @@ -0,0 +1,60 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more contributor license agree

[GitHub] [tvm] masahi commented on a diff in pull request #13710: [TIR] Fix dtype mismatch error due to LetStmt

2023-01-06 Thread GitBox
masahi commented on code in PR #13710: URL: https://github.com/apache/tvm/pull/13710#discussion_r1063891653 ## src/tir/ir/data_type_rewriter.cc: ## @@ -107,6 +107,26 @@ Stmt DataTypeLegalizer::VisitStmt_(const AttrStmtNode* op) { return StmtExprMutator::VisitStmt_(op); }

[GitHub] [tvm] mkatanbaf commented on a diff in pull request #13514: [microTVM] tuning on micro targets with meta-schedule

2023-01-06 Thread GitBox
mkatanbaf commented on code in PR #13514: URL: https://github.com/apache/tvm/pull/13514#discussion_r1063873670 ## python/tvm/contrib/micro/meta_schedule/test_autotune_ms.py: ## @@ -0,0 +1,181 @@ +# Licensed to the Apache Software Foundation (ASF) under one Review Comment: do

[GitHub] [tvm] mkatanbaf commented on a diff in pull request #13514: [microTVM] tuning on micro targets with meta-schedule

2023-01-06 Thread GitBox
mkatanbaf commented on code in PR #13514: URL: https://github.com/apache/tvm/pull/13514#discussion_r1063873557 ## python/tvm/contrib/micro/meta_schedule/test_autotune_ms.py: ## @@ -0,0 +1,181 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor

[GitHub] [tvm] Mousius commented on pull request #13706: Add Name Transforms for Rust style

2023-01-06 Thread GitBox
Mousius commented on PR #13706: URL: https://github.com/apache/tvm/pull/13706#issuecomment-1374245307 @tvm-bot rerun -- 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] tvm-bot commented on pull request #13719: [Hexagon] Add hexagon user DMA intrins for tensorization

2023-01-06 Thread GitBox
tvm-bot commented on PR #13719: URL: https://github.com/apache/tvm/pull/13719#issuecomment-1374243803 Thanks for contributing to TVM! Please refer to the contributing guidelines https://tvm.apache.org/docs/contribute/ for useful information and tips. Please request code reviews from

[GitHub] [tvm] nverke opened a new pull request, #13719: [Hexagon] Add hexagon user DMA intrins for tensorization

2023-01-06 Thread GitBox
nverke opened a new pull request, #13719: URL: https://github.com/apache/tvm/pull/13719 Added some intrins for user dma on hexagon. @adstraw -- 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] tvm-bot commented on pull request #13718: [Fix,AutoScheduler] Handle 0-dim buffers in featurization

2023-01-06 Thread GitBox
tvm-bot commented on PR #13718: URL: https://github.com/apache/tvm/pull/13718#issuecomment-1374237307 Thanks for contributing to TVM! Please refer to the contributing guidelines https://tvm.apache.org/docs/contribute/ for useful information and tips. Please request code reviews from

[GitHub] [tvm] tkonolige opened a new pull request, #13718: [Fix,AutoScheduler] Handle 0-dim buffers in featurization

2023-01-06 Thread GitBox
tkonolige opened a new pull request, #13718: URL: https://github.com/apache/tvm/pull/13718 0-dim buffers get lowered to a 0-dim buffer and a 1-dim buffer decl. We explicitly check for this case and return a stride of 1. @merrymercy @vinx13 -- This is an automated message from the

[tvm] 01/01: Update nrfjprog script and LLVM script

2023-01-06 Thread mehrdadh
This is an automated email from the ASF dual-hosted git repository. mehrdadh pushed a commit to branch ci/update_cortexm_ubuntu20 in repository https://gitbox.apache.org/repos/asf/tvm.git commit a6bc1719dd93e828b08b48e4552191fc3a207076 Author: Mehrdad Hessar AuthorDate: Fri Jan 6 15:02:23 2023 -

[tvm] branch ci/update_cortexm_ubuntu20 created (now a6bc1719dd)

2023-01-06 Thread mehrdadh
This is an automated email from the ASF dual-hosted git repository. mehrdadh pushed a change to branch ci/update_cortexm_ubuntu20 in repository https://gitbox.apache.org/repos/asf/tvm.git at a6bc1719dd Update nrfjprog script and LLVM script This branch includes the following new commits:

[GitHub] [tvm] Mousius commented on pull request #13717: [Tests] Replace pytest.main with tvm.testing.main

2023-01-06 Thread GitBox
Mousius commented on PR #13717: URL: https://github.com/apache/tvm/pull/13717#issuecomment-1374229958 As promised @mehrdadh 😸 I'm fairly sure this is going to fail with some `undefined tvm`. Out of interest do we know why we don't encourage people running tests directly using the `pytest`

[GitHub] [tvm] tvm-bot commented on pull request #13717: [Tests] Replace pytest.main with tvm.testing.main

2023-01-06 Thread GitBox
tvm-bot commented on PR #13717: URL: https://github.com/apache/tvm/pull/13717#issuecomment-1374228814 Thanks for contributing to TVM! Please refer to the contributing guidelines https://tvm.apache.org/docs/contribute/ for useful information and tips. Please request code reviews from

[GitHub] [tvm] Mousius opened a new pull request, #13717: [Tests] Replace pytest.main with tvm.testing.main

2023-01-06 Thread GitBox
Mousius opened a new pull request, #13717: URL: https://github.com/apache/tvm/pull/13717 This ensures that if you want to run a specific test script then at least it's reasonably consistent and as people copy test files they'll use the new function :smile_cat: -- This is an automated mes

[GitHub] [tvm] Mousius commented on a diff in pull request #13704: Add support for named outputs in MLF archive

2023-01-06 Thread GitBox
Mousius commented on code in PR #13704: URL: https://github.com/apache/tvm/pull/13704#discussion_r1063854160 ## tests/python/unittest/test_micro_model_library_format.py: ## @@ -632,5 +632,89 @@ def test_multiple_relay_modules_aot_graph(): assert metadata["version"] == _GENE

[GitHub] [tvm] mehrdadh commented on a diff in pull request #13704: Add support for named outputs in MLF archive

2023-01-06 Thread GitBox
mehrdadh commented on code in PR #13704: URL: https://github.com/apache/tvm/pull/13704#discussion_r1063849396 ## tests/python/unittest/test_micro_model_library_format.py: ## @@ -632,5 +632,89 @@ def test_multiple_relay_modules_aot_graph(): assert metadata["version"] == _GEN

[GitHub] [tvm] masahi commented on a diff in pull request #13710: [TIR] Fix dtype mismatch error due to LetStmt

2023-01-06 Thread GitBox
masahi commented on code in PR #13710: URL: https://github.com/apache/tvm/pull/13710#discussion_r1063847366 ## src/tir/ir/data_type_rewriter.cc: ## @@ -107,6 +107,26 @@ Stmt DataTypeLegalizer::VisitStmt_(const AttrStmtNode* op) { return StmtExprMutator::VisitStmt_(op); }

[GitHub] [tvm] vinx13 commented on a diff in pull request #13710: [TIR] Fix dtype mismatch error due to LetStmt

2023-01-06 Thread GitBox
vinx13 commented on code in PR #13710: URL: https://github.com/apache/tvm/pull/13710#discussion_r1063845468 ## src/tir/ir/data_type_rewriter.cc: ## @@ -107,6 +107,26 @@ Stmt DataTypeLegalizer::VisitStmt_(const AttrStmtNode* op) { return StmtExprMutator::VisitStmt_(op); }

[GitHub] [tvm] tkonolige commented on issue #10317: Auto-cc subscription tracker

2023-01-06 Thread GitBox
tkonolige commented on issue #10317: URL: https://github.com/apache/tvm/issues/10317#issuecomment-1374196548 roofline @tkonolige -- 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

[GitHub] [tvm] tvm-bot commented on pull request #13716: [Fix,Roofline] Fix roofline handling of multiple peak flops

2023-01-06 Thread GitBox
tvm-bot commented on PR #13716: URL: https://github.com/apache/tvm/pull/13716#issuecomment-1374195734 Thanks for contributing to TVM! Please refer to the contributing guidelines https://tvm.apache.org/docs/contribute/ for useful information and tips. Please request code reviews from

[GitHub] [tvm] masahi commented on a diff in pull request #13710: [TIR] Fix dtype mismatch error due to LetStmt

2023-01-06 Thread GitBox
masahi commented on code in PR #13710: URL: https://github.com/apache/tvm/pull/13710#discussion_r1063844872 ## src/tir/ir/data_type_rewriter.cc: ## @@ -107,6 +107,26 @@ Stmt DataTypeLegalizer::VisitStmt_(const AttrStmtNode* op) { return StmtExprMutator::VisitStmt_(op); }

[GitHub] [tvm] tkonolige opened a new pull request, #13716: [Fix,Roofline] Fix roofline handling of multiple peak flops

2023-01-06 Thread GitBox
tkonolige opened a new pull request, #13716: URL: https://github.com/apache/tvm/pull/13716 In the switch to multiple possible peakflops measurement, the logic to add all of them was skipped. Instead only the last was added. @AndrewZhaoLuo -- This is an automated message from the A

[GitHub] [tvm] masahi commented on a diff in pull request #13710: [TIR] Fix dtype mismatch error due to LetStmt

2023-01-06 Thread GitBox
masahi commented on code in PR #13710: URL: https://github.com/apache/tvm/pull/13710#discussion_r1063842929 ## src/tir/ir/data_type_rewriter.cc: ## @@ -107,6 +107,26 @@ Stmt DataTypeLegalizer::VisitStmt_(const AttrStmtNode* op) { return StmtExprMutator::VisitStmt_(op); }

[GitHub] [tvm] masahi commented on a diff in pull request #13710: [TIR] Fix dtype mismatch error due to LetStmt

2023-01-06 Thread GitBox
masahi commented on code in PR #13710: URL: https://github.com/apache/tvm/pull/13710#discussion_r1063842517 ## src/tir/ir/data_type_rewriter.cc: ## @@ -107,6 +107,26 @@ Stmt DataTypeLegalizer::VisitStmt_(const AttrStmtNode* op) { return StmtExprMutator::VisitStmt_(op); }

[GitHub] [tvm] vinx13 commented on a diff in pull request #13710: [TIR] Fix dtype mismatch error due to LetStmt

2023-01-06 Thread GitBox
vinx13 commented on code in PR #13710: URL: https://github.com/apache/tvm/pull/13710#discussion_r1063841351 ## src/tir/ir/data_type_rewriter.cc: ## @@ -107,6 +107,26 @@ Stmt DataTypeLegalizer::VisitStmt_(const AttrStmtNode* op) { return StmtExprMutator::VisitStmt_(op); }

[tvm] branch main updated (21d7968b61 -> 123f1f5e2c)

2023-01-06 Thread kparzysz
This is an automated email from the ASF dual-hosted git repository. kparzysz pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/tvm.git from 21d7968b61 [microTVM] Fix MacOS build with USE_MICRO=ON (#13711) add 123f1f5e2c [tir] Add line level debug info (#13

[GitHub] [tvm] kparzysz-quic merged pull request #13012: [tir] Add line level debug info

2023-01-06 Thread GitBox
kparzysz-quic merged PR #13012: URL: https://github.com/apache/tvm/pull/13012 -- 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.ap

[GitHub] [tvm] masahi commented on a diff in pull request #13710: [TIR] Fix dtype mismatch error due to LetStmt

2023-01-06 Thread GitBox
masahi commented on code in PR #13710: URL: https://github.com/apache/tvm/pull/13710#discussion_r106383 ## src/tir/ir/data_type_rewriter.cc: ## @@ -107,6 +107,26 @@ Stmt DataTypeLegalizer::VisitStmt_(const AttrStmtNode* op) { return StmtExprMutator::VisitStmt_(op); }

[GitHub] [tvm] masahi commented on a diff in pull request #13710: [TIR] Fix dtype mismatch error due to LetStmt

2023-01-06 Thread GitBox
masahi commented on code in PR #13710: URL: https://github.com/apache/tvm/pull/13710#discussion_r1063815756 ## src/tir/ir/data_type_rewriter.cc: ## @@ -107,6 +107,26 @@ Stmt DataTypeLegalizer::VisitStmt_(const AttrStmtNode* op) { return StmtExprMutator::VisitStmt_(op); }

[GitHub] [tvm] guberti commented on a diff in pull request #13627: [docs] Add "Open with Colab" button to documentation

2023-01-06 Thread GitBox
guberti commented on code in PR #13627: URL: https://github.com/apache/tvm/pull/13627#discussion_r1063794348 ## gallery/how_to/work_with_microtvm/install_zephyr.rst: ## @@ -0,0 +1,34 @@ +Install the Prerequisites + + +.. code-block:: bash Review

[GitHub] [tvm] guberti commented on a diff in pull request #13627: [docs] Add "Open with Colab" button to documentation

2023-01-06 Thread GitBox
guberti commented on code in PR #13627: URL: https://github.com/apache/tvm/pull/13627#discussion_r1063793894 ## gallery/how_to/work_with_microtvm/micro_train.py: ## @@ -71,7 +60,7 @@ # # .. code-block:: bash # -# %%bash +# %%shell Review Comment: I've adde

[GitHub] [tvm] Mousius commented on pull request #13704: Add support for named outputs in MLF archive

2023-01-06 Thread GitBox
Mousius commented on PR #13704: URL: https://github.com/apache/tvm/pull/13704#issuecomment-1374120637 @tvm-bot rerun -- 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] guberti commented on a diff in pull request #13627: [docs] Add "Open with Colab" button to documentation

2023-01-06 Thread GitBox
guberti commented on code in PR #13627: URL: https://github.com/apache/tvm/pull/13627#discussion_r1063765019 ## gallery/how_to/work_with_microtvm/micro_aot.py: ## @@ -30,16 +30,40 @@ or on Zephyr platform on a microcontroller/board supported by Zephyr. """ +#

[GitHub] [tvm] guberti commented on a diff in pull request #13627: [docs] Add "Open with Colab" button to documentation

2023-01-06 Thread GitBox
guberti commented on code in PR #13627: URL: https://github.com/apache/tvm/pull/13627#discussion_r1063760366 ## gallery/how_to/work_with_microtvm/install_dependencies.rst: ## @@ -0,0 +1,33 @@ +.. Licensed to the Apache Software Foundation (ASF) under one +or more contributo

[GitHub] [tvm] guberti commented on a diff in pull request #13627: [docs] Add "Open with Colab" button to documentation

2023-01-06 Thread GitBox
guberti commented on code in PR #13627: URL: https://github.com/apache/tvm/pull/13627#discussion_r1063759973 ## gallery/how_to/work_with_microtvm/micro_tflite.py: ## @@ -26,109 +26,36 @@ """ ## -# .. note::

[GitHub] [tvm] guberti commented on a diff in pull request #13627: [docs] Add "Open with Colab" button to documentation

2023-01-06 Thread GitBox
guberti commented on code in PR #13627: URL: https://github.com/apache/tvm/pull/13627#discussion_r1063756803 ## gallery/how_to/work_with_microtvm/micro_aot.py: ## @@ -139,6 +162,12 @@ "board": BOARD, "serial_number": SERIAL, "config_main_stack_size": 4

[GitHub] [tvm] guberti commented on a diff in pull request #13627: [docs] Add "Open with Colab" button to documentation

2023-01-06 Thread GitBox
guberti commented on code in PR #13627: URL: https://github.com/apache/tvm/pull/13627#discussion_r1063753264 ## docs/conf.py: ## @@ -84,6 +86,148 @@ def git_describe_version(original_version): version = git_describe_version(tvm.__version__) release = version + +def monkey_pa

[GitHub] [tvm] guberti commented on a diff in pull request #13627: [docs] Add "Open with Colab" button to documentation

2023-01-06 Thread GitBox
guberti commented on code in PR #13627: URL: https://github.com/apache/tvm/pull/13627#discussion_r1063753264 ## docs/conf.py: ## @@ -84,6 +86,148 @@ def git_describe_version(original_version): version = git_describe_version(tvm.__version__) release = version + +def monkey_pa

[GitHub] [tvm] guberti commented on a diff in pull request #13627: [docs] Add "Open with Colab" button to documentation

2023-01-06 Thread GitBox
guberti commented on code in PR #13627: URL: https://github.com/apache/tvm/pull/13627#discussion_r1063747387 ## apps/microtvm/pyproject.toml: ## @@ -98,22 +95,23 @@ onnxoptimizer = { version = "==0.2.6", optional = true } onnxruntime = { version = "==1.9.0", optional = true }

[GitHub] [tvm] vinx13 commented on issue #13639: [Bug] cudaFree() error when use meta-schedule

2023-01-06 Thread GitBox
vinx13 commented on issue #13639: URL: https://github.com/apache/tvm/issues/13639#issuecomment-1374047527 I also passed test on CUDA 12.0 (I updated target to rtx-3070 on my machine) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Gi

[GitHub] [tvm] areusch commented on pull request #13012: [tir] Add line level debug info

2023-01-06 Thread GitBox
areusch commented on PR #13012: URL: https://github.com/apache/tvm/pull/13012#issuecomment-1374024135 @tvm-bot rerun -- 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] ashutosh-arm commented on issue #13715: [Bug] Fix mprofile gemm schedules for aligned memory accesses and gotos

2023-01-06 Thread GitBox
ashutosh-arm commented on issue #13715: URL: https://github.com/apache/tvm/issues/13715#issuecomment-1374009403 cc @Mousius Can you please take a look at the issue and add any additional concerns here? -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [tvm] ashutosh-arm opened a new issue, #13715: [Bug] Fix mprofile gemm schedules for aligned memory accesses and gotos

2023-01-06 Thread GitBox
ashutosh-arm opened a new issue, #13715: URL: https://github.com/apache/tvm/issues/13715 Fix https://github.com/apache/tvm/commit/9af76f061893cc609c3586567a21220426653b86 that's part of https://github.com/apache/tvm/pull/13363 removes CMSIS-NN dependencies from mprofile schedules, instruct

[GitHub] [tvm] Mousius commented on pull request #13714: [WIP] Test CI for cross-build

2023-01-06 Thread GitBox
Mousius commented on PR #13714: URL: https://github.com/apache/tvm/pull/13714#issuecomment-1374000368 @gigiblender I just noticed that the minimal build appeared in a bunch of PRs and has failed without any Jenkins link - example https://github.com/apache/tvm/pull/13706 -- This is an aut

[GitHub] [tvm] Mousius commented on pull request #13363: [microTVM] Replace arm_nnsupportfunctions.h with arm_acle.h

2023-01-06 Thread GitBox
Mousius commented on PR #13363: URL: https://github.com/apache/tvm/pull/13363#issuecomment-1373997208 @mehrdadh I've pushed up a fix for the hanging, it addresses some of the type punning with`memcpy` :smile_cat: see https://github.com/apache/tvm/pull/13363/commits/9af76f061893cc609c3586567

[GitHub] [tvm] vinx13 commented on a diff in pull request #13710: [TIR] Fix dtype mismatch error due to LetStmt

2023-01-06 Thread GitBox
vinx13 commented on code in PR #13710: URL: https://github.com/apache/tvm/pull/13710#discussion_r1063673395 ## src/tir/ir/data_type_rewriter.cc: ## @@ -107,6 +107,26 @@ Stmt DataTypeLegalizer::VisitStmt_(const AttrStmtNode* op) { return StmtExprMutator::VisitStmt_(op); }

[GitHub] [tvm] mehrdadh commented on pull request #13690: [microTVM][Zephyr]Add project files for mlperftiny submission

2023-01-06 Thread GitBox
mehrdadh commented on PR #13690: URL: https://github.com/apache/tvm/pull/13690#issuecomment-1373952690 @tvm-bot rerun -- 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 uns

[GitHub] [tvm-rfcs] tqchen commented on pull request #96: Embedded Rust API RFC

2023-01-06 Thread GitBox
tqchen commented on PR #96: URL: https://github.com/apache/tvm-rfcs/pull/96#issuecomment-1373871671 > C is used for a variety of use-cases and is foundational to much of the software used in larger systems (not only embedded) I am not denying that fact. Assembly language can also be u

[GitHub] [tvm] Lunderberg commented on pull request #13683: [CMake][OpenCL] Remove warning for OpenCL wrapper

2023-01-06 Thread GitBox
Lunderberg commented on PR #13683: URL: https://github.com/apache/tvm/pull/13683#issuecomment-1373756385 @tvm-bot rerun -- 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 u

[GitHub] [tvm] gigiblender commented on pull request #13714: [WIP] Test CI for cross-build

2023-01-06 Thread GitBox
gigiblender commented on PR #13714: URL: https://github.com/apache/tvm/pull/13714#issuecomment-1373716431 > @gigiblender is this PR related to why every other PR has suddenly failed the build? @Mousius I am not sure I see a correlation --- Could you please share the build error? I t

[GitHub] [tvm] Mousius commented on pull request #13714: [WIP] Test CI for cross-build

2023-01-06 Thread GitBox
Mousius commented on PR #13714: URL: https://github.com/apache/tvm/pull/13714#issuecomment-1373699430 @gigiblender is this PR related to why every other PR has suddenly failed the build? -- This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [tvm-rfcs] Mousius commented on pull request #96: Embedded Rust API RFC

2023-01-06 Thread GitBox
Mousius commented on PR #96: URL: https://github.com/apache/tvm-rfcs/pull/96#issuecomment-1373696783 > I also stated reasonings on why the interface_c name was a OK choice under that the context of C, because C is a language that is mostly used for embedded space -- rust do not have that sa

[GitHub] [tvm-rfcs] tqchen commented on pull request #96: Embedded Rust API RFC

2023-01-06 Thread GitBox
tqchen commented on PR #96: URL: https://github.com/apache/tvm-rfcs/pull/96#issuecomment-1373465521 That particular RFC you were referring to is about C interface and not rust. I also stated reasonings on why the interface_c name was a OK choice under that the context of C, because C

  1   2   >