Skip to content

Update Vitis AI EP to support multiple DPU targets through provider options #6690

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.

Sign up for GitHub

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

Jump to bottom
Merged
skottmckay merged 4 commits into from
Jun 3, 2021
Merged

Update Vitis AI EP to support multiple DPU targets through provider options #6690

skottmckay merged 4 commits into from
Jun 3, 2021

Conversation

@jtuyls
Copy link
Contributor

@jtuyls jtuyls commented Feb 15, 2021

Update the Vitis AI execution provider to support multiple DPU targets, specifically the Zynq DPUCZDX8G target (on arm64).

For documentation, do I need to create a separate PR on the gh-pages branch?

@jtuyls jtuyls requested a review from a team as a code owner February 15, 2021 11:46
@ghost
Copy link

ghost commented Feb 15, 2021 •
edited by ghost
Loading

CLA assistant check
All CLA requirements met.

@faxu
Copy link
Contributor

faxu commented Feb 17, 2021

@jtuyls, yes please submit a PR for doc update on gh-pages branch.

@jywu-msft
Copy link
Member

jywu-msft commented Feb 24, 2021

/azp run orttraining-linux-ci-pipeline,orttraining-mac-ci-pipeline,orttraining-linux-gpu-ci-pipeline,centos7_cpu,Linux CPU Minimal Build E2E CI Pipeline,Linux Nuphar CI Pipeline,MacOS NoContribops CI Pipeline,Linux OpenVINO CI Pipeline,orttraining-distributed

@jywu-msft
Copy link
Member

jywu-msft commented Feb 24, 2021

/azp run Linux CPU CI Pipeline,Linux CPU x64 NoContribops CI Pipeline,Linux GPU CI Pipeline,Linux GPU TensorRT CI Pipeline,MacOS CI Pipeline,MacOS NoContribops CI Pipeline,Windows CPU CI Pipeline,Windows GPU CI Pipeline,Windows GPU TensorRT CI Pipeline

@azure-pipelines
Copy link

azure-pipelines bot commented Feb 24, 2021

Azure Pipelines successfully started running 8 pipeline(s), but failed to run 1 pipeline(s).

@azure-pipelines
Copy link

azure-pipelines bot commented Feb 24, 2021

Azure Pipelines successfully started running 9 pipeline(s).

@jywu-msft
Copy link
Member

jywu-msft commented Feb 25, 2021

/azp run orttraining-amd-gpu-ci-pipeline

@azure-pipelines
Copy link

azure-pipelines bot commented Feb 25, 2021

Azure Pipelines successfully started running 1 pipeline(s).

@hariharans29
Copy link
Member

hariharans29 commented Apr 12, 2021

/azp run orttraining-ortmodule, /azp run orttraining-ortmodule-distributed

@azure-pipelines
Copy link

azure-pipelines bot commented Apr 12, 2021

Azure Pipelines successfully started running 1 pipeline(s).

@hariharans29
Copy link
Member

hariharans29 commented Apr 12, 2021

Hi @jywu-msft : Can you please review this PR ?

@hariharans29
Copy link
Member

hariharans29 commented Apr 12, 2021

/azp run orttraining-ortmodule-distributed

@azure-pipelines
Copy link

azure-pipelines bot commented Apr 12, 2021

Azure Pipelines successfully started running 1 pipeline(s).

@Elonaever
Copy link

Elonaever commented Apr 22, 2021

so can I use the onnxruntime with DPUCVDX8G in the Versal AI Core board?

@jtuyls
Copy link
Contributor Author

jtuyls commented Apr 22, 2021

@Elonaever No, that DPU won't be supported yet. The specific DPU's that will be enabled are: DPUCZDX8G (Zynq) and DPUCAHX8H (U50/U280). However, the changes in this PR make it easier to add more DPU's later on.

skottmckay
skottmckay previously approved these changes Apr 28, 2021
View reviewed changes
@jtuyls jtuyls mentioned this pull request May 26, 2021
Closed
@jtuyls
Copy link
Contributor Author

jtuyls commented Jun 2, 2021 •
edited
Loading

@skottmckay I resolved the conflicts with master. Could you have another look at this PR?

skottmckay
skottmckay approved these changes Jun 3, 2021
View reviewed changes
Copy link
Contributor

@skottmckay skottmckay left a comment

Choose a reason for hiding this comment

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

:shipit:

@skottmckay
Copy link
Contributor

skottmckay commented Jun 3, 2021

/azp run Linux CPU CI Pipeline,Linux CPU Minimal Build E2E CI Pipeline,Linux CPU x64 NoContribops CI Pipeline,Linux GPU CI Pipeline,Linux GPU TensorRT CI Pipeline,Linux Nuphar CI Pipeline,Linux OpenVINO CI Pipeline

@skottmckay
Copy link
Contributor

skottmckay commented Jun 3, 2021

/azp run MacOS CI Pipeline,MacOS NoContribops CI Pipeline,Windows CPU CI Pipeline,Windows GPU CI Pipeline,Windows GPU TensorRT CI Pipeline,Windows WebAssembly CI Pipeline

@skottmckay
Copy link
Contributor

skottmckay commented Jun 3, 2021

/azp run orttraining-amd-gpu-ci-pipeline,orttraining-linux-ci-pipeline,orttraining-linux-gpu-ci-pipeline,orttraining-ortmodule,orttraining-ortmodule-distributed

@azure-pipelines
Copy link

azure-pipelines bot commented Jun 3, 2021

Azure Pipelines successfully started running 6 pipeline(s).

@azure-pipelines
Copy link

azure-pipelines bot commented Jun 3, 2021

Azure Pipelines successfully started running 5 pipeline(s).

@azure-pipelines
Copy link

azure-pipelines bot commented Jun 3, 2021

Azure Pipelines successfully started running 7 pipeline(s).

@skottmckay skottmckay merged commit 3bb780d into microsoft:master Jun 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@skottmckay skottmckay skottmckay approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@jtuyls @faxu @jywu-msft @hariharans29 @Elonaever @skottmckay @jornt-xilinx