Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[CI Problem] Failure to import requests, caffe in ci_arm, ci_cpu #12790

Closed
ashutosh-arm opened this issue Sep 15, 2022 · 1 comment · Fixed by #12813
Closed

[CI Problem] Failure to import requests, caffe in ci_arm, ci_cpu #12790

ashutosh-arm opened this issue Sep 15, 2022 · 1 comment · Fixed by #12813
Assignees
Labels
type:ci Relates to TVM CI infrastructure

Comments

@ashutosh-arm
Copy link
Contributor

The issue being reported was seen on local machine with latest docker images. It is not visible in CI yet as the docker hub images are little old. PR #12663 introduced virtual environments where python and its dependencies are installed. Some of the linting issues were fixed here: #12775. However, some tests fail due to error while importing some packages.

  1. import requests fails to import this module. It affects many test suites as the below search result suggests.
 ./3rdparty/vta-hw/apps/deploy/python_deploy.py:25:import requests
./ci/scripts/github_skipped_tests_comment.py:27:import requests
./python/tvm/contrib/download.py:58:            import requests
./tests/micro/arduino/test_utils.py:20:import requests
./tests/micro/zephyr/test_utils.py:30:import requests
./vta/tutorials/frontend/deploy_classification.py:43:import argparse, json, os, requests, sys, time
  1. import caffe fails to load caffe. The reason being caffe installer still creates soft link outside of virtual env.
    ln -s ${PYCAFFE_ROOT}/caffe /usr/local/lib/python3.7/dist-packages/caffe
    . It affects all the caffe tests in TVM.

@leandron @areusch @driazati

@ashutosh-arm ashutosh-arm added the needs-triage PRs or issues that need to be investigated by maintainers to find the right assignees to address it label Sep 15, 2022
@areusch areusch self-assigned this Sep 15, 2022
@areusch areusch added type:ci Relates to TVM CI infrastructure and removed needs-triage PRs or issues that need to be investigated by maintainers to find the right assignees to address it labels Sep 15, 2022
@ashutosh-arm
Copy link
Contributor Author

Raised the PR to fix import issues: #12813

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:ci Relates to TVM CI infrastructure
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants