Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[core] Tensor names sync on change not after Result node validation #28945

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

praasz
Copy link
Contributor

@praasz praasz commented Feb 12, 2025

Details:

  • Introducing special tensor descriptor for Result caused that changing tensor names on connected input required node re-validation to see updated names. Now the names will be updated without Result node validation.
  • Add dedicated tensor wrappers when connected to Result tensor to distinguish generic input from Parameter input.
  • Add dev API function to copy names on result node which skip if input is parameter.
  • Split tensor descriptor and OV tensor tests to separate files.

Tickets:

@praasz praasz requested review from a team as code owners February 12, 2025 06:52
@github-actions github-actions bot added category: Core OpenVINO Core (aka ngraph) category: IR FE OpenVINO IR v10 / v11 FrontEnd category: extensions OpenVINO Extensibility Mechanism for custom operations category: CPP API OpenVINO CPP API bindings labels Feb 12, 2025
@praasz praasz added this to the 2025.1 milestone Feb 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: Core OpenVINO Core (aka ngraph) category: CPP API OpenVINO CPP API bindings category: extensions OpenVINO Extensibility Mechanism for custom operations category: IR FE OpenVINO IR v10 / v11 FrontEnd
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants