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 2022/08/16 10:36:54 UTC
[GitHub] [tvm] ekalda opened a new pull request, #12456: [CI] Set USE_CMSISNN and USE_ETHOSU off in task_config_build_cpu.sh
ekalda opened a new pull request, #12456:
URL: https://github.com/apache/tvm/pull/12456
The dependencies for these have moved into ci_cortexm Docker
image, so there is not much point in building them for ci_cpu as we
can't run the associated 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.
To unsubscribe, e-mail: commits-unsubscribe@tvm.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [tvm] ekalda commented on pull request #12456: [CI] Set USE_CMSISNN and USE_ETHOSU off in task_config_build_cpu.sh
Posted by GitBox <gi...@apache.org>.
ekalda commented on PR #12456:
URL: https://github.com/apache/tvm/pull/12456#issuecomment-1220738998
Hi @areusch , sorry I've had very busy last few days, but I'm on it now - I looked into this and the problem is that we still install Vela in ci_cpu (https://github.com/apache/tvm/blob/main/docker/Dockerfile.ci_cpu#L129) despite moving Ethos-U into ci_cortexm and we use the presence of Vela as a proxy when deciding whether to run a test or not if it is outside of `test_ethosu` folder. As a result of this, there are some Ethos-U based tests in tvmc testing which are failing because ```pytest.importorskip("ethosu.vela")``` can't do it's job (https://github.com/apache/tvm/blob/main/tests/python/driver/tvmc/test_compiler.py#L456) .
So I think I'll park this change for a bit and propose a docker image change and once that is done, we should be able to merge this twoliner patch :)
--
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] ekalda commented on pull request #12456: [CI] Set USE_CMSISNN and USE_ETHOSU off in task_config_build_cpu.sh
Posted by GitBox <gi...@apache.org>.
ekalda commented on PR #12456:
URL: https://github.com/apache/tvm/pull/12456#issuecomment-1216462329
cc @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.
To unsubscribe, e-mail: commits-unsubscribe@tvm.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [tvm] ekalda commented on pull request #12456: [CI] Set USE_CMSISNN and USE_ETHOSU off in task_config_build_cpu.sh
Posted by GitBox <gi...@apache.org>.
ekalda commented on PR #12456:
URL: https://github.com/apache/tvm/pull/12456#issuecomment-1245625785
@driazati @areusch @leandron it's passing the CI now after the upgrade to the Docker image, so should be fine to merge.
--
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] areusch commented on pull request #12456: [CI] Set USE_CMSISNN and USE_ETHOSU off in task_config_build_cpu.sh
Posted by GitBox <gi...@apache.org>.
areusch commented on PR #12456:
URL: https://github.com/apache/tvm/pull/12456#issuecomment-1220025860
thanks @ekalda ! looks like we are still running one of those tests in CPU somehow, maybe need to make sure that one got moved to arm and then remove it from CPU.
--
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] ekalda commented on pull request #12456: [CI] Set USE_CMSISNN and USE_ETHOSU off in task_config_build_cpu.sh
Posted by GitBox <gi...@apache.org>.
ekalda commented on PR #12456:
URL: https://github.com/apache/tvm/pull/12456#issuecomment-1243522663
@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 unsubscribe, e-mail: commits-unsubscribe@tvm.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [tvm] driazati merged pull request #12456: [CI] Set USE_CMSISNN and USE_ETHOSU off in task_config_build_cpu.sh
Posted by GitBox <gi...@apache.org>.
driazati merged PR #12456:
URL: https://github.com/apache/tvm/pull/12456
--
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