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

[VitisAI] Add profiler interface for vitisai #23032

Merged
merged 3 commits into from
Dec 16, 2024

Conversation

tianfang-fafafa
Copy link
Contributor

Description

Add common interfaces for vitis ep profiler.

Motivation and Context

Vitis ep can collect and record api and kernel timestamps in file when onnxruntime '-p' is enabled.

@tianfang-fafafa
Copy link
Contributor Author

tianfang-fafafa commented Dec 6, 2024

hi, @jywu-msft Could you help to review this pr, please?

@xlnx-yuxiz
Copy link

Hi, @jywu-msft , could you please help review this PR for vitisai?

@jywu-msft
Copy link
Member

/azp run Linux CPU CI Pipeline, Linux CPU Minimal Build E2E CI Pipeline, Linux GPU CI Pipeline, Linux GPU TensorRT CI Pipeline, Linux OpenVINO CI Pipeline, MacOS CI Pipeline, ONNX Runtime Web CI Pipeline, onnxruntime-binary-size-checks-ci-pipeline, Linux QNN CI Pipeline

@jywu-msft
Copy link
Member

/azp run Windows ARM64 QNN CI Pipeline, Windows CPU CI Pipeline, Windows GPU TensorRT CI Pipeline, Windows x64 QNN CI Pipeline, orttraining-linux-ci-pipeline, orttraining-linux-gpu-ci-pipeline

@jywu-msft
Copy link
Member

/azp run Big Models, Linux Android Emulator QNN CI Pipeline, Windows GPU CUDA CI Pipeline, Windows GPU DML CI Pipeline, Windows GPU Doc Gen CI Pipeline

Copy link

Azure Pipelines successfully started running 4 pipeline(s).

Copy link

Azure Pipelines successfully started running 5 pipeline(s).

Copy link

Azure Pipelines successfully started running 9 pipeline(s).

@tianfang-fafafa
Copy link
Contributor Author

hi @jywu-msft , all checks have passed, is it suitable for merging?

@jywu-msft jywu-msft merged commit a4eb8f2 into microsoft:main Dec 16, 2024
77 checks passed
guschmue pushed a commit that referenced this pull request Dec 16, 2024
### Description
<!-- Describe your changes. -->
Add common interfaces for vitis ep profiler.


### Motivation and Context
<!-- - Why is this change required? What problem does it solve?
- If it fixes an open issue, please link to the issue here. -->
Vitis ep can collect and record api and kernel timestamps in file when
onnxruntime '-p' is enabled.
@tianfang-fafafa
Copy link
Contributor Author

@jywu-msft thanks a lot

guschmue pushed a commit that referenced this pull request Dec 20, 2024
### Description
<!-- Describe your changes. -->
Add common interfaces for vitis ep profiler.


### Motivation and Context
<!-- - Why is this change required? What problem does it solve?
- If it fixes an open issue, please link to the issue here. -->
Vitis ep can collect and record api and kernel timestamps in file when
onnxruntime '-p' is enabled.
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