[ONNX] Align behavior of ONNX Frontend operator ReduceMean-11, 13, 18 with original framework

[rghvsh]

Details:

• Align behavior of ONNX Frontend operator ReduceMean-11, 13, 18 with original framework

Tickets:

• Closes [Good First Issue]: Align behavior of ONNX Frontend operator ReduceMean-11, 13, 18 with original framework #20556
  CVS-143344, CVS-125493

[gkrivor]

Hi, I refactored reduce.cpp and reduce.hpp. I think it should be more simple to add a new functionality.

Please, rebase after merge.

#23429

[gkrivor]

Hi @rghvsh ,

Please, take a look on #23475 as a reference

[rghvsh]

Hi @gkrivor! I had a few questions:

do we need to add opset 11 to reduce.hpp
do we need to add opset 13 to reduce.cpp and reduce.hpp
in ops_bridge do I explicitly mention opset 11,13,18 or type this
register_operator("ReduceMean", VersionRange{1, 17}, op::set_1::reduce_mean);
register_operator("ReduceLogSum", VersionRange::since(18), op::set_1::reduce_mean);

Thank You

[github-actions]

This PR will be closed in a week because of 2 weeks of no activity.

[rghvsh]

Hi @gkrivor @mitruska @onnx do you have any suggestions

[rghvsh]

Hi @gkrivor make changes today

[gkrivor]

You should add a model for Opset-18. Please, take a look on reduce_max_18.prototxt as an example.

Major thing you should reflect - provide an "axes" as an input instead of an attribute

[gkrivor]

@rghvsh as you can see, your change is very important and fix a lot of failing tests:
https://github.com/openvinotoolkit/openvino/actions/runs/8875797101/job/24366553495?pr=23148
Let's complete it asap. You need to remove failing cases from https://github.com/openvinotoolkit/openvino/blob/master/src/frontends/onnx/tests/tests_python/test_backend.py

[mitruska]

The import code LGTM, but the tests are missing.

[mitruska]

The test model has been added, but there is no unit test code for it, so it's not tested.
For each added model there should be corresponding test in openvino/src/frontends/onnx/tests/onnx_import.in.cpp.
Also please keep the names of the onnx test models files aligned (snake_case).

Example:

openvino/src/frontends/onnx/tests/onnx_import.in.cpp

Lines 1018 to 1019 in 59c54cb

	OPENVINO_TEST(${BACKEND_NAME}, onnx_model_reduce_log_sum_18_axes_as_input) {
	auto model = convert_model("reduce_log_sum_18_axes_as_input.onnx");

[gkrivor]

@rghvsh could you add some tests and remove test which are working after your changes (take a look on my previous comment)?

[gkrivor]

@rghvsh any updates?

[gkrivor]

build_jenkins

[gkrivor]

build_jenkins

[gkrivor]

build_jenkins

[gkrivor]

build_jenkins

[gkrivor]

build_jenkins