Turn on Compute library testing in CI for AArch64 #8291
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
added 3 commits
June 21, 2021 11:10
This pull request turns on compute library testing in CI by 1. Handling import errors in Compute Library Integration. 2. Setting the configuration to the right path for ACL. This handles import errors for packages in Compute library integration. This pull request allows for the AArch64 CI to pick up native compute library testing and tests the operators being offloaded at runtime.
* Move to using pre-built ACL binaries for ci_arm * Fixup the path for installation to be /opt/acl as it originally was. * Fix up the issues with paths. Once this is done ci_arm will need to be rebuilt though will continue to work seamlessly.
lhutton1
approved these changes
Jun 22, 2021
Contributor
lhutton1
left a comment
lhutton1
left a comment
There was a problem hiding this comment.
LGTM, feel free to ignore small suggestion
cc @areusch @mbrookhart, I believe this change requires a container update?
| try: | ||
| import tvm.relay.testing.tf as tf_testing | ||
| except ImportError: | ||
| pytest.skip("Missing TF Support") |
Contributor
There was a problem hiding this comment.
Suggested change
| pytest.skip("Missing TF Support") | |
| pytest.skip("Missing Tflite Support") |
26 tasks
Contributor
|
I'm rebuilding CI containers this week, I'll include this in that. |
mbrookhart
approved these changes
Jun 22, 2021
Contributor
ylc
pushed a commit
to ylc/tvm
that referenced
this pull request
Sep 29, 2021
* Turn on Compute library testing in CI. This pull request turns on compute library testing in CI by 1. Handling import errors in Compute Library Integration. 2. Setting the configuration to the right path for ACL. This handles import errors for packages in Compute library integration. This pull request allows for the AArch64 CI to pick up native compute library testing and tests the operators being offloaded at runtime. * Fix typo * Fix up use of ubuntu_install_arm_compute_lib.sh in Dockerfile.ci_arm * Move to using pre-built ACL binaries for ci_arm * Fixup the path for installation to be /opt/acl as it originally was. * Fix up the issues with paths. Once this is done ci_arm will need to be rebuilt though will continue to work seamlessly.
zxy844288792
pushed a commit
to zxy844288792/tvm
that referenced
this pull request
Mar 4, 2022
* Turn on Compute library testing in CI. This pull request turns on compute library testing in CI by 1. Handling import errors in Compute Library Integration. 2. Setting the configuration to the right path for ACL. This handles import errors for packages in Compute library integration. This pull request allows for the AArch64 CI to pick up native compute library testing and tests the operators being offloaded at runtime. * Fix typo * Fix up use of ubuntu_install_arm_compute_lib.sh in Dockerfile.ci_arm * Move to using pre-built ACL binaries for ci_arm * Fixup the path for installation to be /opt/acl as it originally was. * Fix up the issues with paths. Once this is done ci_arm will need to be rebuilt though will continue to work seamlessly.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This pull request turns on compute library testing in CI by
This pull request allows for the AArch64 CI to pick up native
compute library testing and tests the operators being offloaded at
runtime.
@mbaret @d-smirnov @lhutton1 - could I get a review please ?