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/graph_iterator_proto.cpp

@@ -312,11 +312,12 @@ GraphIteratorProto::GraphIteratorProto(GraphIteratorProto* parent, const GraphPr
     m_model = parent->m_model;
 }
 
-void GraphIteratorProto::initialize(const std::string& path) {
-    m_model_dir = std::make_shared<std::string>(ov::util::get_directory(path).string());
+void GraphIteratorProto::initialize_from_path(const std::filesystem::path& path) {
+    m_model_dir = std::make_shared<std::string>(ov::util::path_to_string(ov::util::get_directory(path)));
+    const auto path_string = ov::util::path_to_string(path);
     try {
         std::ifstream model_file(path, std::ios::binary | std::ios::in);
-        FRONT_END_GENERAL_CHECK(model_file && model_file.is_open(), "Could not open the file: \"", path, "\"");
+        FRONT_END_GENERAL_CHECK(model_file && model_file.is_open(), "Could not open the file: \"", path_string, "\"");
 
         m_model = std::make_shared<ModelProto>();
         FRONT_END_GENERAL_CHECK(m_model->ParseFromIstream(&model_file), "Model can't be parsed");
@@ -337,9 +338,13 @@ void GraphIteratorProto::initialize(const std::string& path) {
     }
 }
 
+void GraphIteratorProto::initialize(const std::string& path) {
+    initialize_from_path(ov::util::make_path(path));
+}
+
 #ifdef OPENVINO_ENABLE_UNICODE_PATH_SUPPORT
 void GraphIteratorProto::initialize(const std::wstring& path) {
-    initialize(ov::util::wstring_to_string(path));
+    initialize_from_path(ov::util::make_path(path));
 }
 #endif  // OPENVINO_ENABLE_UNICODE_PATH_SUPPORT
 

src/frontends/onnx/frontend/src/core/graph_iterator_proto.hpp

@@ -4,6 +4,8 @@
 #pragma once
 #include <onnx/onnx_pb.h>
 
+#include <filesystem>
+
 #include <openvino/frontend/graph_iterator.hpp>
 
 #include "openvino/frontend/onnx/decoder.hpp"
@@ -150,6 +152,9 @@ class GraphIteratorProto : public ov::frontend::onnx::GraphIterator {
     /// \param name Name of tensor
     /// \param owner Returns real owner of the tensor
     std::shared_ptr<DecoderProtoTensor> get_tensor(const std::string& name, GraphIteratorProto** owner);
+
+private:
+    void initialize_from_path(const std::filesystem::path& path);
 };
 
 }  // namespace onnx

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 upcast decoder-provided floats here to align the behavior of OpExtension with the 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,29 @@ 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 +111,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;
     }