onnxruntime icon indicating copy to clipboard operation
onnxruntime copied to clipboard

[VitisAI] remove unused header

Open BoarQing opened this issue 1 year ago • 1 comments

Description

Removed unused headers

Motivation and Context

This would cause compile error on machine that didn't install nlohmann.

BoarQing avatar Aug 28 '24 06:08 BoarQing

@jywu-msft Could you review this when you have time? No functionality is changed.

BoarQing avatar Aug 29 '24 02:08 BoarQing

/azp run Windows ARM64 QNN CI Pipeline,Windows x64 QNN CI Pipeline,Windows CPU CI Pipeline,Windows GPU CUDA CI Pipeline,Windows GPU DML CI Pipeline,Windows GPU Doc Gen CI Pipeline,Windows GPU TensorRT CI Pipeline,ONNX Runtime Web CI Pipeline,Linux CPU CI Pipeline,Linux CPU Minimal Build E2E CI Pipeline

tianleiwu avatar Sep 03 '24 20:09 tianleiwu

/azp run Linux GPU CI Pipeline,Linux GPU TensorRT CI Pipeline,Linux OpenVINO CI Pipeline,Linux QNN CI Pipeline,MacOS CI Pipeline,orttraining-amd-gpu-ci-pipeline,orttraining-linux-ci-pipeline,orttraining-linux-gpu-ci-pipeline,orttraining-ortmodule-distributed,onnxruntime-binary-size-checks-ci-pipeline

tianleiwu avatar Sep 03 '24 20:09 tianleiwu

/azp run Big Models,Linux Android Emulator QNN CI Pipeline,Android CI Pipeline,iOS CI Pipeline,ONNX Runtime React Native CI Pipeline

tianleiwu avatar Sep 03 '24 20:09 tianleiwu

Azure Pipelines successfully started running 5 pipeline(s).

azure-pipelines[bot] avatar Sep 03 '24 20:09 azure-pipelines[bot]

Azure Pipelines successfully started running 10 pipeline(s).

azure-pipelines[bot] avatar Sep 03 '24 20:09 azure-pipelines[bot]

Azure Pipelines successfully started running 10 pipeline(s).

azure-pipelines[bot] avatar Sep 03 '24 20:09 azure-pipelines[bot]

/azp run ONNX Runtime Web CI Pipeline

tianleiwu avatar Sep 04 '24 03:09 tianleiwu

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar Sep 04 '24 03:09 azure-pipelines[bot]

can you merge latest main to address the Linux OpenVINO CI Pipeline failure?

jywu-msft avatar Sep 04 '24 18:09 jywu-msft

can you merge latest main to address the Linux OpenVINO CI Pipeline failure?

I just merged it. @jywu-msft

BoarQing avatar Sep 05 '24 01:09 BoarQing

/azp run Windows ARM64 QNN CI Pipeline,Windows x64 QNN CI Pipeline,Windows CPU CI Pipeline,Windows GPU CUDA CI Pipeline,Windows GPU DML CI Pipeline,Windows GPU Doc Gen CI Pipeline,Windows GPU TensorRT CI Pipeline,ONNX Runtime Web CI Pipeline,Linux CPU CI Pipeline,Linux CPU Minimal Build E2E CI Pipeline

tianleiwu avatar Sep 05 '24 03:09 tianleiwu

/azp run Linux GPU CI Pipeline,Linux GPU TensorRT CI Pipeline,Linux OpenVINO CI Pipeline,Linux QNN CI Pipeline,MacOS CI Pipeline,orttraining-amd-gpu-ci-pipeline,orttraining-linux-ci-pipeline,orttraining-linux-gpu-ci-pipeline,orttraining-ortmodule-distributed,onnxruntime-binary-size-checks-ci-pipeline

tianleiwu avatar Sep 05 '24 03:09 tianleiwu

/azp run Big Models,Linux Android Emulator QNN CI Pipeline,Android CI Pipeline,iOS CI Pipeline,ONNX Runtime React Native CI Pipeline

tianleiwu avatar Sep 05 '24 03:09 tianleiwu

Azure Pipelines successfully started running 10 pipeline(s).

azure-pipelines[bot] avatar Sep 05 '24 03:09 azure-pipelines[bot]

Azure Pipelines successfully started running 5 pipeline(s).

azure-pipelines[bot] avatar Sep 05 '24 03:09 azure-pipelines[bot]

Azure Pipelines successfully started running 10 pipeline(s).

azure-pipelines[bot] avatar Sep 05 '24 03:09 azure-pipelines[bot]

@tianleiwu @jywu-msft Can we merge this?

BoarQing avatar Sep 05 '24 14:09 BoarQing