[Build] C++ API calling fail: error C2280: 'Ort::Value::Value(const Ort::Value &)' : attempt to reference a deleted function

[lucasjinreal]

Describe the issue

error C2280: 'Ort::Value::Value(const Ort::Value &)' : attempt to reference a deleted function

Urgency

Very urgent

Target platform

windows

Build script

noio

Error / output

error C2280: 'Ort::Value::Value(const Ort::Value &)' : attempt to reference a deleted function

Here is the c++ caling code:

 std::vector<Ort::Value> ort_inputs;
  for (int i = 0; i < inputNames.size(); ++i) {
    auto ort_value = Ort::Value::CreateTensor<float>(
        memoryInfo, static_cast<float *>(inputs[i].data), inputs[i].get_size(),
        inputShapes[i].data(), inputShapes[i].size());
    assert(ort_value.IsTensor());
    ort_inputs.emplace_back(ort_value);
  }

try {
    std::vector<Ort::Value> ort_outputs;
    ort_outputs = session.Run(
        Ort::RunOptions{nullptr}, inputNames.data(), ort_inputs.data(),
        inputNames.size(), outputNames.data(), outputNames.size());
}

can not build

Visual Studio Version

2019

GCC / Compiler Version

No response

[baijumeswani]

Would you please try to create the ort value in place in the vector?

std::vector<Ort::Value> ort_inputs;
  for (int i = 0; i < inputNames.size(); ++i) {
    ort_inputs.emplace_back(Ort::Value::CreateTensor<float>(
        memoryInfo, static_cast<float *>(inputs[i].data),
        inputs[i].get_size(), inputShapes[i].data(), inputShapes[i].size()));
  }

...

[lucasjinreal]

@baijumeswani Hi, I have tried, but not work, the problem caused is second part code:

try {
    std::vector<Ort::Value> ort_outputs;
    ort_outputs = session.Run(
        Ort::RunOptions{nullptr}, inputNames.data(), ort_inputs.data(),
        inputNames.size(), outputNames.data(), outputNames.size());
}

I don't know where caused the above exception.

[RyanUnderhill]

My guess is you're trying to copy the vector of outputs returned by Run() into ort_outputs.

Try this:

std::vector<Ort::Value> ort_outputs = session.Run(...)

[lucasjinreal]

@RyanUnderhill hi, if I am not dosing so, how can I get the outputs? Doesn't they are identical?

[RyanUnderhill]

True, those actually should be identical and the return value should just be a move, so no copying is needed.

Can you see what line the error is happening on? And what version of C++ you're compiling with?

[lucasjinreal]

@RyanUnderhill hello, I am using MSVC 2019 version, the error messages comes from onnxruntime header so I don't know eactly which line caused issue.

[RyanUnderhill]

@RyanUnderhill hello, I am using MSVC 2019 version, the error messages comes from onnxruntime header so I don't know eactly which line caused issue.

I mean, can you paste your code with line numbers and the output of MSVC 2019 with the error information. It should give you a line number of the error in your code.

Somewhere in your code, you're telling the compiler to copy an Ort::Value, which is prohibited.

[lucasjinreal]

@RyanUnderhill unfortrunately MSVC didnt give me which line (my code) got above errors.

[iPengXPro]

I got the same problem,did you solve this problem?

[RyanUnderhill]

In your case do you know what line the problem happened on?

[raisinglc03]

I got the same problem. Although I solved it, I have no idea why...

The following code return "error: use of deleted function ‘Ort::Value::Value(const Ort::Value&)"
{ ::new((void *)__p) _Up(std::forward<_Args>(__args)...); }

vector<Ort::Value> outputtensors;
auto outputtensor = Ort::Value::CreateTensor<float>( 
                memoryInfo, tmpTensorValue.data(), tmpOutTensorSize, tmpOutDim.data(), tmpOutDim.size()
            );
outputtensors.push_back(outputtensor);

I put CreateTensor inside push_back and then it work ...

vector<Ort::Value> outputtensors;
outputtensors.push_back( Ort::Value::CreateTensor<float>( 
                memoryInfo, tmpTensorValue.data(), tmpOutTensorSize, tmpOutDim.data(), tmpOutDim.size()
            );