When will onnxruntime make it available to get nodeunit inputs correctly for QLinearConcat about onnxruntime HOT 4 OPEN

Do you have a model where this causes an issue?

from onnxruntime.

Sure, the model is downloaded from onnx model zoo:(https://github.com/onnx/models/blob/main/validated/vision/classification/densenet-121/model/densenet-12-int8.onnx).
The unit test is from path onnxruntime/onnxruntime/test/contrib_ops/qlinear_concat_test.cc(https://github.com/microsoft/onnxruntime/blob/main/onnxruntime/test/contrib_ops/qlinear_concat_test.cc)

from onnxruntime.

I don't think the lack of support makes a difference in this case.

QLinearConcat is an internal operator, so we would expect it to be handled by our internal execution providers. In that scenario, the fact that the zp/scale values aren't nicely provided by the NodeUnit doesn't change anything as nothing (AFAIK) is trying to read the zp and scale from the NodeUnit as we have static kernels for CPU/CUDA/DML execution providers to handle QLinearConcat.

Really for the model to be in the ONNX repo it should be in QDQ format (DequantizeLinear on each input with a Concat node and a QuantizeLinear on the output) instead of having been saved using internal onnxruntime operators. With that model format any execution provider could convert the set of nodes into a quantized concat operation, and it would want the zp/scale for the inputs to be conveniently available via the NodeUnit. That's a different NodeUnit constructor though and not the path with the TODO comment.

from onnxruntime.