Skip to content

Modify CANN EP to align with the EP API refactor and fix CANN CI #16490

Merged
jywu-mysoft merged 1 commit into
microsoft:mainfrom
zhangsibo1129:fix_cann_ci
Jul 6, 2023
Merged

Modify CANN EP to align with the EP API refactor and fix CANN CI #16490
jywu-mysoft merged 1 commit into
microsoft:mainfrom
zhangsibo1129:fix_cann_ci

Conversation

@zhangsibo1129

Copy link
Copy Markdown
Contributor

Modify CANN EP CANNExecutionProvider::CreatePreferredAllocators, CANNExecutionProvider::CreateCannAllocator
to align with the EP API refactor and fix CANN CI for #15833 (comment) in this PR

@zhangsibo1129

Copy link
Copy Markdown
Contributor Author

@microsoft-github-policy-service agree

@zhangsibo1129

Copy link
Copy Markdown
Contributor Author

I fixed the CANN EP build for this #15833 (comment), @jywu-msft would you take a look at this PR. Thanks a lot!

thank you, @zhangsibo1129! @jslhcl can you help take a look?

@jslhcl could you help review the code? Thank you!

@jywu-mysoft

Copy link
Copy Markdown
Member

/azp run Linux CPU CI Pipeline, Linux CPU Minimal Build E2E CI Pipeline, Linux GPU CI Pipeline, Linux GPU TensorRT CI Pipeline, Linux Nuphar CI Pipeline, Linux OpenVINO CI Pipeline, MacOS CI Pipeline, ONNX Runtime Web CI Pipeline, Windows CPU CI Pipeline, Windows GPU CI Pipeline

@jywu-mysoft

Copy link
Copy Markdown
Member

/azp run Windows GPU TensorRT CI Pipeline, onnxruntime-binary-size-checks-ci-pipeline, onnxruntime-python-checks-ci-pipeline, orttraining-linux-ci-pipeline, orttraining-linux-gpu-ci-pipeline, orttraining-ortmodule-distributed

@jywu-mysoft

Copy link
Copy Markdown
Member

/azp run Linux QNN CI Pipeline, Windows ARM64 QNN CI Pipeline

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 2 pipeline(s).

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 5 pipeline(s).

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 9 pipeline(s).

@jslhcl jslhcl left a comment

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.

:shipit:

@zhangsibo1129

Copy link
Copy Markdown
Contributor Author

@jslhcl @jywu-msft Thank you for your review, helping fix the CANN CI.

@jywu-mysoft jywu-mysoft merged commit 180292f into microsoft:main Jul 6, 2023
@zhangsibo1129 zhangsibo1129 deleted the fix_cann_ci branch July 17, 2023 02:44
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.

3 participants