Enable GraphIterator interface by default

.github/workflows/job_cxx_unit_tests.yml

@@ -140,11 +140,19 @@ jobs:
           ${{ env.SOURCE_COMMAND }} ${{ env.SETUPVARS }}
           ${{ env.INSTALL_TEST_DIR }}/paddle_tests --gtest_filter="PaddleFuzzyOpTest*" --gtest_print_time=1 --gtest_output=xml:${{ env.INSTALL_TEST_DIR }}/TEST-PaddleFuzzyOpTests.xml
 
-      - name: ONNX frontend tests
+      - name: ONNX frontend tests (GraphIterator default)
         if: ${{ fromJSON(inputs.affected-components).ONNX_FE.test && runner.arch != 'ARM64' }} # Ticket for macOS ARM64: 122663, for Linux ARM64: 126280, 153161
         run: |
           ${{ env.SOURCE_COMMAND }} ${{ env.SETUPVARS }}
-          ${{ env.INSTALL_TEST_DIR }}/ov_onnx_frontend_tests --gtest_print_time=1 --gtest_filter=-*IE_GPU* --gtest_output=xml:${{ env.INSTALL_TEST_DIR }}/TEST-ONNXFrontend.xml
+          ${{ env.INSTALL_TEST_DIR }}/ov_onnx_frontend_tests --gtest_print_time=1 --gtest_filter=-*IE_GPU* --gtest_output=xml:${{ env.INSTALL_TEST_DIR }}/TEST-ONNXFrontend-GraphIteratorEnabled.xml
+
+      - name: ONNX frontend tests (ONNX_ITERATOR=0)
+        if: ${{ fromJSON(inputs.affected-components).ONNX_FE.test && runner.arch != 'ARM64' }} # Ticket for macOS ARM64: 122663, for Linux ARM64: 126280, 153161
+        env:
+          ONNX_ITERATOR: "0"
+        run: |
+          ${{ env.SOURCE_COMMAND }} ${{ env.SETUPVARS }}
+          ${{ env.INSTALL_TEST_DIR }}/ov_onnx_frontend_tests --gtest_print_time=1 --gtest_filter=-*IE_GPU* --gtest_output=xml:${{ env.INSTALL_TEST_DIR }}/TEST-ONNXFrontend-GraphIteratorDisabled.xml
 
       - name: TensorFlow Common frontend tests
         if: fromJSON(inputs.affected-components).TF_FE.test ||

.github/workflows/job_python_unit_tests.yml

@@ -111,14 +111,26 @@ jobs:
       # Tests
       #
 
-      - name: Python ONNX operators tests
+      - name: Python ONNX operators tests (GraphIterator default)
         if: (fromJSON(inputs.affected-components).Python_API.test ||
              fromJSON(inputs.affected-components).ONNX_FE.test) &&
              runner.arch != 'ARM64' # Ticket: 123325
         run: |
           # Skip test_onnx/test_zoo_models and test_onnx/test_backend due to long execution time - ONNX Model Zoo tests are run separately
           python3 -m pytest -sv ${INSTALL_TEST_DIR}/onnx -k 'not cuda' \
-            --junitxml=${INSTALL_TEST_DIR}/TEST-onnx_frontend.xml \
+            --junitxml=${INSTALL_TEST_DIR}/TEST-onnx_frontend.graph_iterator_enabled.xml \
+            --ignore=${INSTALL_TEST_DIR}/onnx/test_python/test_zoo_models.py
+
+      - name: Python ONNX operators tests (ONNX_ITERATOR=0)
+        if: (fromJSON(inputs.affected-components).Python_API.test ||
+             fromJSON(inputs.affected-components).ONNX_FE.test) &&
+             runner.arch != 'ARM64' # Ticket: 123325
+        env:
+          ONNX_ITERATOR: "0"
+        run: |
+          # Skip test_onnx/test_zoo_models and test_onnx/test_backend due to long execution time - ONNX Model Zoo tests are run separately
+          python3 -m pytest -sv ${INSTALL_TEST_DIR}/onnx -k 'not cuda' \
+            --junitxml=${INSTALL_TEST_DIR}/TEST-onnx_frontend.graph_iterator_disabled.xml \
             --ignore=${INSTALL_TEST_DIR}/onnx/test_python/test_zoo_models.py
 
       - name: OVC unit tests

src/frontends/onnx/frontend/src/core/node.cpp

@@ -221,7 +221,28 @@ Subgraph Node::Impl::get_attribute_value(const std::string& name) const {
 
 template <>
 ov::Any Node::get_attribute_value(const std::string& name) const {
-    return get_attribute(name).get_any();
+    if (m_pimpl != nullptr) {
+        return get_attribute(name).get_any();
+    } else if (m_decoder != nullptr) {
+        if (!m_decoder->has_attribute(name)) {
+            throw error::node::UnknownAttribute{this->get_name(), name};
+        }
+        auto attribute_value = m_decoder->get_attribute(name);
+        // We do upcast for decoder-provided floats here to align behavior of OpExtension with legacy approach and keep
+        // both code paths identical.
+        if (attribute_value.is<float>()) {
+            return static_cast<double>(attribute_value.as<float>());
+        } else if (attribute_value.is<std::vector<float>>()) {
+            const auto& float_values = attribute_value.as<std::vector<float>>();
+            std::vector<double> double_values(float_values.size());
+            std::transform(float_values.begin(), float_values.end(), double_values.begin(), [](float v) {
+                return static_cast<double>(v);
+            });
+            return double_values;
+        }
+        return attribute_value;
+    }
+    FRONT_END_NOT_IMPLEMENTED(get_attribute_value);
 }
 
 template <>

src/frontends/onnx/frontend/src/frontend.cpp

@@ -17,8 +17,12 @@
 #    include <google/protobuf/stubs/logging.h>
 #endif
 
+#include <algorithm>
+#include <cctype>
+#include <cstdlib>
 #include <fstream>
 #include <sstream>
+#include <string>
 
 #include "core/graph_iterator_proto.hpp"
 #include "input_model.hpp"
@@ -48,6 +52,30 @@ using ::ONNX_NAMESPACE::ModelProto;
 using ::ONNX_NAMESPACE::Version;
 
 namespace {
+bool is_graph_iterator_disabled(const char* env_value) {
+    if (env_value == nullptr) {
+        return false;
+    }
+
+    std::string value(env_value);
+    value.erase(std::remove_if(value.begin(), value.end(), [](unsigned char ch) {
+                    return std::isspace(ch) != 0;
+                }),
+                value.end());
+    std::transform(value.begin(),
+                   value.end(),
+                   value.begin(),
+                   [](unsigned char ch) {
+                       return static_cast<char>(std::tolower(ch));
+                   });
+
+    return value.empty() || value == "0" || value == "false" || value == "off" || value == "disable";
+}
+
+bool is_graph_iterator_enabled() {
+    return !is_graph_iterator_disabled(std::getenv("ONNX_ITERATOR"));
+}
+
 // !!! Experimental feature, it may be changed or removed in the future !!!
 void enumerate_constants(const std::shared_ptr<ov::Model>& model) {
     const auto& operations = model->get_ordered_ops();
@@ -84,7 +112,7 @@ ONNX_FRONTEND_C_API void* get_front_end_data() {
 }
 
 ov::frontend::InputModel::Ptr FrontEnd::load_impl(const std::vector<ov::Any>& variants) const {
-    const bool gi_enabled = std::getenv("ONNX_ITERATOR") != nullptr;
+    const bool gi_enabled = is_graph_iterator_enabled();
     if (variants.empty()) {
         return nullptr;
     }