Skip to content

[WIP] NNAPI EP Update - #1540

Merged
Changming Sun (snnn) merged 15 commits into
microsoft:masterfrom
daquexian:android_ci
Aug 2, 2019
Merged

[WIP] NNAPI EP Update#1540
Changming Sun (snnn) merged 15 commits into
microsoft:masterfrom
daquexian:android_ci

Conversation

@daquexian

Copy link
Copy Markdown
Contributor

Description:

  • Enable Android CI --- The yml path is android-x86_64-crosscompile-ci-pipeline.yml instead of android-arm64-crosscompile-ci-pipeline.yml now

  • Support NNAPI EP in perftest

@daquexian
daquexian (daquexian) requested a review from a team as a code owner August 1, 2019 10:34
@daquexian

daquexian (daquexian) commented Aug 1, 2019

Copy link
Copy Markdown
Contributor Author

Need help:

--enable_onnx_tests requires some model files. I have not figured out where I can get them. Log:

[ RUN ] ONNXModelsTests/ONNXModelsTest.LoadFromFile/0
/media/sdb1/daquexian/repos/onnxruntime/onnxruntime/test/ir/onnx_model_test.cc:118: Failure
Value of: Model::Load(GetModelFileName(), model).IsOK()
Actual: false
Expected: true
[ FAILED ] ONNXModelsTests/ONNXModelsTest.LoadFromFile/0, where GetParam() = "bvlc_alexnet" (0 ms)

jobs:
- job: Android_CI
pool:
vmImage: 'macOS-10.14'

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For building the android package on Mac? Why do we need to test it? Why not using Linux instead?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It seems macos has better Android emulator support: https://github.com/microsoft/azure-pipelines-image-generation/blob/master/images/macos/macos-10.14-Readme.md, and since we are doing cross-compile via official Android NDK, macos or linux doesn't matter a lot.

@snnn

Copy link
Copy Markdown
Contributor

--enable_onnx_tests

You may disable it first. I'll take care it later.

@snnn

Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 21 pipeline(s), but failed to run 1 pipeline(s).

@snnn
Changming Sun (snnn) merged commit 93cb29f into microsoft:master Aug 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants