onnxruntime
onnxruntime copied to clipboard
[Nuget] Add netstandard* to buildTransitive folders (microsoft#17010)
The netstandard2.0 folder was added to the build folder, but not the buildTransitive folder. This means that packages like Microsoft.ML.OnnxRuntime aren't being included as transitive build dependencies.
For example, if I have a nuget of my MyLibrary which has a dependency on Microsoft.ML.OnnxRuntime and someone adds MyLibrary to their application their application would not pull in the Microsoft.ML.OnnxRuntime nuget. This manifests when the application is run and there is a onnxruntime not found error.
NOTE: I was NOT able to build packages so I couldn't test this out. Any feedback would be much appreciated.
fixes #17010
/azp run Windows ARM64 QNN CI Pipeline,Windows x64 QNN CI Pipeline,Windows CPU CI Pipeline,Windows GPU CI Pipeline,Windows GPU TensorRT CI Pipeline,ONNX Runtime Web CI Pipeline,Linux CPU CI Pipeline,Linux CPU Minimal Build E2E CI Pipeline,Linux GPU CI Pipeline,Linux GPU TensorRT CI Pipeline
/azp run 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,Big Models
Azure Pipelines successfully started running 9 pipeline(s).
/azp run Windows ARM64 QNN CI Pipeline,Windows x64 QNN CI Pipeline,Windows CPU CI Pipeline,Windows GPU CI Pipeline,Windows GPU TensorRT CI Pipeline,ONNX Runtime Web CI Pipeline,Linux CPU CI Pipeline,Linux CPU Minimal Build E2E CI Pipeline,Linux GPU CI Pipeline,Linux GPU TensorRT CI Pipeline
/azp run orttraining-amd-gpu-ci-pipeline
Azure Pipelines successfully started running 1 pipeline(s).
Azure Pipelines successfully started running 10 pipeline(s).
"python format" is not triggered. I will close and reopen to trigger it.
Python format pipeline failed. Please run lintrunner -a to fix format. To set up lintrunner locally, see https://github.com/microsoft/onnxruntime/blob/main/docs/Coding_Conventions_and_Standards.md#linting
BTW: Why are not all folders from build also in buildTransitive?
@decriptor: It would be awesome if your commit would succeed. Do you think you can make it work?
@msedi Sorry, I've moved on to much different projects. I think the only thing left to fix the python formatting? If you'd like to take over the changes and fix the python formatting that'd be great.
https://github.com/microsoft/onnxruntime/pull/19242#issuecomment-2041218278
I'm going to close this out, but anyone is welcome to take these changes and finish the work.