Skip to content

[Refactoring] Fixed common spelling typos in core components#34841

Open
Lagmator22 wants to merge 3 commits intoopenvinotoolkit:masterfrom
Lagmator22:fix/core-typos
Open

[Refactoring] Fixed common spelling typos in core components#34841
Lagmator22 wants to merge 3 commits intoopenvinotoolkit:masterfrom
Lagmator22:fix/core-typos

Conversation

@Lagmator22
Copy link
Copy Markdown
Contributor

Details:

  • This PR fixes several common spelling errors (e.g. "paramater", "avaliable", "suport", "overrided", "occurence") scattered across the core C/C++ headers and tests.

Tickets:

  • N/A (Minor typo fixes)

AI Assistance:

  • AI assistance used: yes
  • If yes, summarize how AI was used and what human validation was performed:
    Used an AI assistant to run regex searches across the codebase for common spelling mistakes. Manually verified every single occurrence in the IDE to ensure they were legitimate typos and that no third-party submodule dependencies were affected.

@Lagmator22 Lagmator22 requested review from a team as code owners March 22, 2026 07:56
@Lagmator22 Lagmator22 requested review from Copilot and whitneyfoster and removed request for a team March 22, 2026 07:56
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR corrects several spelling typos across core C/C++ headers, tests, and documentation to improve readability and professionalism without changing runtime behavior.

Changes:

  • Fixed repeated “occurence” → “occurrence” in a functional test.
  • Corrected common typos in core plugin headers and inference code comments (e.g., “suport”, “transfered”, “Paramater”).
  • Updated API docs and error strings to fix “paramater/overrided/avaliable” typos.

Reviewed changes

Copilot reviewed 11 out of 11 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/tests/functional/plugin/conformance/subgraphs_dumper/tests/cache/op_cache.cpp Fixes typo in test comment (“occurrence”).
src/plugins/intel_cpu/src/nodes/tensoriterator.h Fixes typo in struct field Doxygen comment (“transferred”).
src/plugins/intel_cpu/src/nodes/subgraph.h Fixes typo in comment (“Parameter”).
src/inference/src/single_file_storage.cpp Fixes typo in internal tracking comment (“support”).
src/core/tests/type_prop/transpose.cpp Fixes typo in test doc comment (“parameter”).
src/core/shape_inference/include/element_visitor.hpp Fixes typo in API doc comment (“parameter”).
src/core/include/openvino/op/op.hpp Fixes typo in macro comment (“overridden”).
src/common/snippets/tests/src/lir_comparator.cpp Fixes typo in returned error message (“overridden”).
src/bindings/python/src/pyopenvino/graph/node_output.hpp Fixes typo in binding comment (“available”).
src/bindings/c/include/openvino/c/ov_core.h Fixes typo in C API doc comment (“parameter”).
src/bindings/c/docs/api_overview.md Fixes typo in C API overview docs (“parameter”).

@sys-openvino-ci sys-openvino-ci added the ExternalPR External contributor label Mar 22, 2026
@github-actions github-actions Bot added category: inference OpenVINO Runtime library - Inference category: Core OpenVINO Core (aka ngraph) category: IE Tests OpenVINO Test: plugins and common category: CPU OpenVINO CPU plugin category: Python API OpenVINO Python bindings category: C API OpenVINO C API bindings category: docs OpenVINO documentation category: CPP API OpenVINO CPP API bindings category: snippets labels Mar 22, 2026
@rkazants
Copy link
Copy Markdown
Collaborator

build_jenkins

@Lagmator22
Copy link
Copy Markdown
Contributor Author

Hi @rkazants, I have fixed the backslash alignment in op.hpp to resolve the clang-format CI failure. Ready for build_jenkins.

@Lagmator22
Copy link
Copy Markdown
Contributor Author

@whitneyfoster, a gentle ping could you please rerun the tests please and review the changes for approval?

@praasz
Copy link
Copy Markdown
Contributor

praasz commented Mar 27, 2026

build_jenkins

@Lagmator22
Copy link
Copy Markdown
Contributor Author

@rkazants @praasz thanks for the re-run yesterday! Just checking in, the two internal Jenkins checks (cijjenkins and npu_trigger_ubuntu22_release) still show as failing, but I assume those are running on infra I can't access. Is there anything else needed from my side, or is this ready to merge? Happy to update the branch if that helps.

@praasz praasz added this to the 2026.2 milestone Mar 31, 2026
@Lagmator22
Copy link
Copy Markdown
Contributor Author

@praasz The branch is now 406 commits behind master with a conflict in src/inference/src/single_file_storage.cpp the conflict is just a comment-typo (suport -> support). Since this PR is purely cosmetic and already has approvals, would you prefer I rebase and re-trigger CI, or can it be merged with conflict resolution on your end? Im fine to do whichever is least disruptive.

@praasz
Copy link
Copy Markdown
Contributor

praasz commented May 5, 2026

build_jenkins

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category: C API OpenVINO C API bindings category: Core OpenVINO Core (aka ngraph) category: CPP API OpenVINO CPP API bindings category: CPU OpenVINO CPU plugin category: docs OpenVINO documentation category: IE Tests OpenVINO Test: plugins and common category: inference OpenVINO Runtime library - Inference category: Python API OpenVINO Python bindings category: snippets ExternalPR External contributor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants