Ryan Hill
Ryan Hill
> Sorry, I have been busy the last few week. From talking to @RyanUnderhill, we are going to use libloading so that you can specify the path to the shared...
I wonder if the arm_compute headers changed, can you try making this change to see if these errors go away: https://github.com/microsoft/onnxruntime/blob/315e00653211b81129b4a8718f74d35afd4a2aaf/onnxruntime/core/providers/acl/nn/conv.cc#L21 After that line, add: `#include "arm_compute/runtime/tensor.h"`
Yep, that error makes sense. There was a recent change to conv_attributes.h renaming the InferOutputShape method to InferPadsAndOutputShape. So just change InferOutputShape to InferPadsAndOutputShape in conv.cc line 108 Looks like...
Looks like the latest armnn libraries had non trivial changes. You're welcome to keep investigating, but it'd be faster for you use a previous version of their library that works....
I'd guess one from 2020? I don't know if they guarantee non breaking API changes within a version but I'd guess v20.11 could work. https://github.com/ARM-software/ComputeLibrary/releases/tag/v20.11 From: https://github.com/ARM-software/ComputeLibrary/tags
It looks like the original owners of that provider are not maintaining it (it was submitted by outsiders). We have added a bunch of similar Arm CPU specific improvements to...
> @skottmckay @edgchen1 Hi, could you please take a look at this PR? BIG thanks. I'm eager to know what should I do to push forward this code. To other...
> We can't require all our customers using C++17. While we can choose which C++ standard to use for our code, we can't make the decision for our customers. Darn,...
/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...
/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