Fix typos in frontends and few obvious issues in onnx and pytorch (#33853)

### Details:
 - *Fix typos in frontends*
 - *Fix obvious problems in ONNX and PyTorch frontends*

---------

Signed-off-by: Maxim Vafin <maxim.vafin@intel.com>

Author: mvafin

src/frontends/common/include/openvino/frontend/exception.hpp

@@ -76,15 +76,15 @@ class FRONTEND_API NotImplementedFailure : public AssertFailure {
 /// \param cond Condition to check
 /// \param ... Additional error message info to be added to the error message via the `<<`
 ///            stream-insertion operator. Note that the expressions here will be evaluated lazily,
-///            i.e., only if the `cond` evalutes to `false`.
+///            i.e., only if the `cond` evaluates to `false`.
 /// \throws ::ov::frontend::GeneralFailure if `cond` is false.
 #define FRONT_END_GENERAL_CHECK(...) OPENVINO_ASSERT_HELPER(::ov::frontend::GeneralFailure, "", __VA_ARGS__)
 
 /// \brief Macro to check whether a boolean condition holds.
 /// \param cond Condition to check
 /// \param ... Additional error message info to be added to the error message via the `<<`
 ///            stream-insertion operator. Note that the expressions here will be evaluated lazily,
-///            i.e., only if the `cond` evalutes to `false`.
+///            i.e., only if the `cond` evaluates to `false`.
 /// \throws ::ov::frontend::InitializationFailure if `cond` is false.
 #define FRONT_END_INITIALIZATION_CHECK(...) \
     OPENVINO_ASSERT_HELPER(::ov::frontend::InitializationFailure, "", __VA_ARGS__)
@@ -93,7 +93,7 @@ class FRONTEND_API NotImplementedFailure : public AssertFailure {
 /// \param cond Condition to check
 /// \param ... Additional error message info to be added to the error message via the `<<`
 ///            stream-insertion operator. Note that the expressions here will be evaluated lazily,
-///            i.e., only if the `cond` evalutes to `false`.
+///            i.e., only if the `cond` evaluates to `false`.
 /// \throws ::ov::frontend::OpConversionFailure if `cond` is false.
 #define FRONT_END_OP_CONVERSION_CHECK(...) OPENVINO_ASSERT_HELPER(::ov::frontend::OpConversionFailure, "", __VA_ARGS__)
 

src/frontends/common/src/frontend.cpp

@@ -73,7 +73,7 @@ std::shared_ptr<Model> FrontEnd::decode(const InputModel::Ptr& model) const {
 
 void FrontEnd::normalize(const std::shared_ptr<Model>& model) const {
     FRONT_END_CHECK_IMPLEMENTED(m_actual, normalize);
-    FRONTEND_CALL_STATEMENT("Normalizing model", m_actual->normalize(model);)
+    FRONTEND_CALL_STATEMENT("Normalizing model", m_actual->normalize(model))
 }
 
 void FrontEnd::add_extension(const std::shared_ptr<ov::Extension>& extension) {
@@ -106,7 +106,7 @@ std::string FrontEnd::get_name() const {
     if (!m_actual) {
         return {};
     }
-    FRONTEND_RETURN_STATEMENT("Getting frontend name", m_actual->get_name();)
+    FRONTEND_RETURN_STATEMENT("Getting frontend name", m_actual->get_name())
 }
 
 void FrontEnd::validate_path(const std::filesystem::path& path) const {

src/frontends/jax/src/utils.hpp

@@ -29,7 +29,7 @@ const std::string& get_jax_prefix();
 /// \param COND Condition to check
 /// \param ... Additional error message info to be added to the error message via the `<<`
 ///            stream-insertion operator. Note that the expressions here will be evaluated lazily,
-///            i.e., only if the `cond` evalutes to `false`.
+///            i.e., only if the `cond` evaluates to `false`.
 /// \throws ::ov::frontend::OpConversionFailure if `cond` is false.
 #ifndef JAX_OP_CONVERSION_CHECK
 #    define JAX_OP_CONVERSION_CHECK(COND, ...) \

src/frontends/onnx/docs/check_supported_ops.py

@@ -63,7 +63,7 @@ def run_in_ci():
         if line.count('|') == 6:
             table_line += 1
         if table_line > 2:
-            row = [cell.strip() for cell in line.split('|')] # Split line by "|" delimeter and remove spaces
+            row = [cell.strip() for cell in line.split('|')] # Split line by "|" delimiter and remove spaces
             domain = row[1]
             if not domain in ops:
                 ops[domain] = {}

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

@@ -46,18 +46,25 @@ inline float get_value(const AttributeProto& attribute) {
 
 template <>
 inline std::vector<float> get_value(const AttributeProto& attribute) {
+#if defined(_MSC_VER)
+#    pragma warning(push)
+#    pragma warning(disable : 4244)
+#endif
     switch (attribute.type()) {
     case AttributeProto_AttributeType::AttributeProto_AttributeType_INT:
         return {static_cast<float>(attribute.i())};
     case AttributeProto_AttributeType::AttributeProto_AttributeType_INTS:
-        return {std::begin(attribute.floats()), std::end(attribute.floats())};
+        return {std::begin(attribute.ints()), std::end(attribute.ints())};
     case AttributeProto_AttributeType::AttributeProto_AttributeType_FLOAT:
         return {attribute.f()};
     case AttributeProto_AttributeType::AttributeProto_AttributeType_FLOATS:
         return {std::begin(attribute.floats()), std::end(attribute.floats())};
     default:
         ONNX_INVALID_ATTR(attribute.type(), "INT, INTS, FLOAT, FLOATS");
     }
+#if defined(_MSC_VER)
+#    pragma warning(pop)
+#endif
 }
 
 template <>

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

@@ -97,13 +97,13 @@ class Graph : public std::enable_shared_from_this<Graph> {
 ///             cache.
 class Subgraph : public Graph {
 public:
-    /// \brief      Subgraph a GraphCache class object.
+    /// \brief      Subgraph holds a GraphCache class object.
     ///
     /// \param[in]  model          The ONNX model object.
     /// \param[in]  parent_graph   The reference to the parent graph.
     Subgraph(const std::shared_ptr<ModelProto>& model, Graph* parent_graph);
 
-    /// \brief      Return nodes which are on the edge the subgraph and the parent graph.
+    /// \brief      Return nodes which are on the edge of the subgraph and the parent graph.
     /// \return     Vector of edge nodes from parent scope.
     const std::vector<ov::Output<ov::Node>> get_inputs_from_parent() const;
 

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

@@ -20,29 +20,29 @@ class GraphCache {
     ///
     /// \note       GraphCache takes ownership of the node.
     ///
-    /// \param[in]  name       The name of node added to the cache.
+    /// \param[in]  name       The name of the node added to the cache.
     /// \param[in]  node       The node added to the cache.
     void emplace_node(const std::string& name, ov::Output<ov::Node>&& node);
 
     /// \brief      Remove node from the cache
     ///
-    /// \param[in]  name       The name of node to be removed
+    /// \param[in]  name       The name of the node to be removed
     void remove_node(const std::string& name);
 
     /// \brief      Get the node from the cache
     ///
-    /// \note       If the node is not found the ov::Exception is thrown.
+    /// \note       If the node is not found, an ov::Exception is thrown.
     ///
     /// \param[in]  name       The name of the node.
     ///
     /// \return     The node named `name`.
     virtual ov::Output<ov::Node> get_node(const std::string& name) const;
 
-    /// \brief      Return true if the node named `name` exist in the cache.
+    /// \brief      Return true if the node named `name` exists in the cache.
     ///
     /// \param[in]  name       The name of the node.
     ///
-    /// \return     true if the node named `name` exist in the cache, false otherwise.
+    /// \return     true if the node named `name` exists in the cache, false otherwise.
     virtual bool contains(const std::string& name) const;
 
     virtual ~GraphCache() = default;

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

@@ -54,7 +54,7 @@ class GraphIteratorProto : public ov::frontend::onnx::GraphIterator {
     GraphIteratorProtoMemoryManagementMode m_mode;
     // This is used for keeping MMAP cache handles
     MappedMemoryHandles m_mmap_cache;
-    // This is used for keeping a readed external data without MMAP
+    // This is used for keeping external data read without MMAP
     LocalStreamHandles m_stream_cache;
     LocalMemoryHandles m_data_holder;
 

src/frontends/onnx/frontend/src/detail/subgraph_extraction.hpp

@@ -30,7 +30,7 @@ struct SubgraphExtractor {
 
     /// \brief Adds new inputs to the graph and connects them to the nodes indicated by
     ///        the provided input edges.
-    /// \param merge_inputs Flag indicating wheteher new inputs are generated for each input edge (false)
+    /// \param merge_inputs Flag indicating whether new inputs are generated for each input edge (false)
     ///                     or there is only one input created for provided edges (true)
     void add_new_inputs(const std::vector<InputEdge>& new_inputs, const bool merge_inputs = false);
 

src/frontends/onnx/frontend/src/edge_mapper.hpp

@@ -29,7 +29,7 @@ class EdgeMapper {
 
     /// \brief Creates an edge mapper based on a GraphProto object.
     ///
-    /// \note If state of graph_proto will be changed, the information from edge mapper
+    /// \note If the state of graph_proto changes, the information from the edge mapper
     ///       is outdated. In such a case the update method should be called.
     ///
     /// \param graph_proto Reference to a GraphProto object.
@@ -39,37 +39,37 @@ class EdgeMapper {
     ///        and an input (input name or input index).
     ///
     /// \note  The node name can be ambiguous (many ONNX nodes can have the same name).
-    ///        In such a case the algorthim tries to match the given node name
+    ///        In such a case the algorithm tries to match the given node name
     ///        with the input name (providing an input index is not enough).
     ///        If a unique edge is found, it will be returned.
     ///        If InputEdge cannot be determined based on parameter values an
-    ///        ov:Exception will be thrown.
+    ///        ov::Exception will be thrown.
     ///
     /// \param node An EditorNode helper structure created based on a node name
     ///             or a node output name.
     ///
-    /// \param input An EditorInput helper structure created based on a input name
-    ///              or a input index.
+    /// \param input An EditorInput helper structure created based on an input name
+    ///              or an input index.
     InputEdge find_input_edge(const EditorNode& node, const EditorInput& input) const;
 
     /// \brief Returns an OutputEdge based on a node (node name or output name)
     ///        and an output (output name or output index).
     ///
     /// \note  The node name can be ambiguous (many ONNX nodes can have the same name).
-    ///        In such a case the algorthim will try to match the given node name
+    ///        In such a case the algorithm will try to match the given node name
     ///        with the output name (providing an output index is not enough).
-    ///        If after such operation a found edge is unique, it is returned.
+    ///        If the found edge is unique after such an operation, it is returned.
     ///        If OutputEdge cannot be determined based on given params an
     ///        ov::Exception is thrown.
     ///
     /// \param node An EditorNode helper structure created based on a node name
     ///             or a node output name.
     ///
-    /// \param output An EditorOutput helper structure created based on a output name
-    ///               or a output index.
+    /// \param output An EditorOutput helper structure created based on an output name
+    ///               or an output index.
     OutputEdge find_output_edge(const EditorNode& node, const EditorOutput& output) const;
 
-    /// \brief Returns an OutputEdge based on a output name.
+    /// \brief Returns an OutputEdge based on an output name.
     ///
     /// \note  The output name guarantees the uniqueness of the edge.
     ///