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 2021/05/20 09:57:38 UTC
[GitHub] [tvm] melsonlai opened a new pull request #8088: [BYOC][NNAPI]: Add testing package to ci_cpu image
melsonlai opened a new pull request #8088:
URL: https://github.com/apache/tvm/pull/8088
This commit adds Android SDK to the ci_cpu image for supporting tests of Android NNAPI BYOC.
See also:
+ [\[RFC\]\[BYOC\] Android NNAPI Integration](https://discuss.tvm.apache.org/t/rfc-byoc-android-nnapi-integration/9072)
+ PR #8076
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] [tvm] mbrookhart commented on pull request #8088: [BYOC][NNAPI]: Add testing package to ci_cpu image
Posted by GitBox <gi...@apache.org>.
mbrookhart commented on pull request #8088:
URL: https://github.com/apache/tvm/pull/8088#issuecomment-873501513
Still working on it :( We've hit lots of incompatible versions and typos in install scripts. See #8177 for details
--
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-unsubscribe@tvm.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [tvm] comaniac commented on pull request #8088: [BYOC][NNAPI]: Add testing package to ci_cpu image
Posted by GitBox <gi...@apache.org>.
comaniac commented on pull request #8088:
URL: https://github.com/apache/tvm/pull/8088#issuecomment-845471519
Accordingly, we will put this PR on hold for now, and @areusch will merge it when the issues of updating container images are resolved.
--
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] [tvm] mbrookhart commented on pull request #8088: [BYOC][NNAPI]: Add testing package to ci_cpu image
Posted by GitBox <gi...@apache.org>.
mbrookhart commented on pull request #8088:
URL: https://github.com/apache/tvm/pull/8088#issuecomment-866315317
Rebuilding containers this week.
Thanks @melsonlai @areusch @comaniac @leandron
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [tvm] mbrookhart merged pull request #8088: [BYOC][NNAPI]: Add testing package to ci_cpu image
Posted by GitBox <gi...@apache.org>.
mbrookhart merged pull request #8088:
URL: https://github.com/apache/tvm/pull/8088
--
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] [tvm] areusch commented on pull request #8088: [BYOC][NNAPI]: Add testing package to ci_cpu image
Posted by GitBox <gi...@apache.org>.
areusch commented on pull request #8088:
URL: https://github.com/apache/tvm/pull/8088#issuecomment-855008724
oh sorry, i was just sorting through PRs now and not reading very well. do we now want to move forward with this PR and rebuild containers again?
--
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] [tvm] areusch commented on pull request #8088: [BYOC][NNAPI]: Add testing package to ci_cpu image
Posted by GitBox <gi...@apache.org>.
areusch commented on pull request #8088:
URL: https://github.com/apache/tvm/pull/8088#issuecomment-845369330
hi all, i added some updates on #7995 to let you know where we are. given the complexity there, i'd prefer if we could keep this PR out of it and then we can update again after that. The CUDA issues encountered on 18.04 are making this container update more challenging than normal.
--
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] [tvm] melsonlai commented on pull request #8088: [BYOC][NNAPI]: Add testing package to ci_cpu image
Posted by GitBox <gi...@apache.org>.
melsonlai commented on pull request #8088:
URL: https://github.com/apache/tvm/pull/8088#issuecomment-873378478
Hi, had the `ci_cpu` container been updated? It's seems that Android SDK is still not there in the Jenkins pipeline.
--
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-unsubscribe@tvm.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [tvm] comaniac commented on pull request #8088: [BYOC][NNAPI]: Add testing package to ci_cpu image
Posted by GitBox <gi...@apache.org>.
comaniac commented on pull request #8088:
URL: https://github.com/apache/tvm/pull/8088#issuecomment-855012106
Sure we could do that. Please merge this PR when you are going to rebuild the container.
--
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] [tvm] melsonlai commented on pull request #8088: [BYOC][NNAPI]: Add testing package to ci_cpu image
Posted by GitBox <gi...@apache.org>.
melsonlai commented on pull request #8088:
URL: https://github.com/apache/tvm/pull/8088#issuecomment-844936205
cc @comaniac @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] [tvm] areusch commented on pull request #8088: [BYOC][NNAPI]: Add testing package to ci_cpu image
Posted by GitBox <gi...@apache.org>.
areusch commented on pull request #8088:
URL: https://github.com/apache/tvm/pull/8088#issuecomment-855008099
@melsonlai @comaniac #7995 has landed, since the CI did not fail before, i'm unsure whether we should retest before merging?
--
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