You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tvm.apache.org by GitBox <gi...@apache.org> on 2020/09/27 10:14:52 UTC

[GitHub] [incubator-tvm] insop opened a new pull request #6569: [DOC] Fix missing te in the code example

insop opened a new pull request #6569:
URL: https://github.com/apache/incubator-tvm/pull/6569


   This PR fixed the code example in this [page](https://tvm.apache.org/docs/deploy/hls.html).
   
   - caused `AttributeError: module ‘tvm’ has no attribute ‘thread_axis’` error
   - Fix based on [this thread](https://discuss.tvm.apache.org/t/attributeerror-module-tvm-has-no-attribute-thread-axis/6606)
   
   Thanks for contributing to TVM!   Please refer to guideline https://tvm.apache.org/docs/contribute/ for useful information and tips. After the pull request is submitted, please request code reviews from [Reviewers](https://github.com/apache/incubator-tvm/blob/master/CONTRIBUTORS.md#reviewers) by @ them in the pull request thread.
   


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [incubator-tvm] comaniac commented on pull request #6569: [DOC] Fix missing te in the code example

Posted by GitBox <gi...@apache.org>.
comaniac commented on pull request #6569:
URL: https://github.com/apache/incubator-tvm/pull/6569#issuecomment-699688170


   You can push a dummy commit like this one: https://coderwall.com/p/vkdekq/git-commit-allow-empty
   
   But please wait until #6572 is merged and the CI on master is green.


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [incubator-tvm] insop commented on pull request #6569: [DOC] Fix missing te in the code example

Posted by GitBox <gi...@apache.org>.
insop commented on pull request #6569:
URL: https://github.com/apache/incubator-tvm/pull/6569#issuecomment-699686737


   > The CI error is irrelevant. Please re-trigger when the master branch is fixed.
   
   Thank you @zhiics , how can I re-trigger The ci tests?
   


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [incubator-tvm] comaniac merged pull request #6569: [DOC] Fix missing te in the code example

Posted by GitBox <gi...@apache.org>.
comaniac merged pull request #6569:
URL: https://github.com/apache/incubator-tvm/pull/6569


   


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [incubator-tvm] insop commented on pull request #6569: [DOC] Fix missing te in the code example

Posted by GitBox <gi...@apache.org>.
insop commented on pull request #6569:
URL: https://github.com/apache/incubator-tvm/pull/6569#issuecomment-699690134


   > You can push a dummy commit like this one: https://coderwall.com/p/vkdekq/git-commit-allow-empty
   > 
   > But please wait until #6572 is merged and the CI on master is green.
   
   Thank you @comaniac .
   Got it, I understand about dummy commit. I asked because some of the Github/Jenkins setup I used, there is a way to trigger a new build without a dummy build.
   Will wait for #6572 .
   


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [incubator-tvm] comaniac commented on pull request #6569: [DOC] Fix missing te in the code example

Posted by GitBox <gi...@apache.org>.
comaniac commented on pull request #6569:
URL: https://github.com/apache/incubator-tvm/pull/6569#issuecomment-699761856


   Thanks @insop @zhiics 


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [incubator-tvm] insop commented on pull request #6569: [DOC] Fix missing te in the code example

Posted by GitBox <gi...@apache.org>.
insop commented on pull request #6569:
URL: https://github.com/apache/incubator-tvm/pull/6569#issuecomment-699697973


   > > You can push a dummy commit like this one: https://coderwall.com/p/vkdekq/git-commit-allow-empty
   > > But please wait until #6572 is merged and the CI on master is green.
   > 
   > Thank you @comaniac .
   > Got it, I understand about dummy commit. I asked because some of the Github/Jenkins setup I used, there is a way to trigger a new build without a dummy build.
   > Will wait for #6572 .
   
   I rebased to the master after #6572 , which triggered the build.


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org