Skip to content

Source id for ov::Tensor#34607

Open
olpipi wants to merge 5 commits intoopenvinotoolkit:masterfrom
olpipi:Tensor_id
Open

Source id for ov::Tensor#34607
olpipi wants to merge 5 commits intoopenvinotoolkit:masterfrom
olpipi:Tensor_id

Conversation

@olpipi
Copy link
Copy Markdown
Contributor

@olpipi olpipi commented Mar 10, 2026

Details:

  • Set/Get source id for ov::Tensor
  • Automatically inherit source id, if tensor is created from AlignedBuffer using new make_tensor func

Tickets:

AI Assistance:

  • AI assistance used: no / yes
  • If yes, summarize how AI was used and what human validation was performed (build/tests/manual checks).

@github-actions github-actions bot added category: inference OpenVINO Runtime library - Inference category: Core OpenVINO Core (aka ngraph) labels Mar 10, 2026
@olpipi olpipi marked this pull request as ready for review March 11, 2026 14:10
@olpipi olpipi requested review from a team as code owners March 11, 2026 14:10
@olpipi olpipi changed the title [WIP] Source id for ov::Tensor Source id for ov::Tensor Mar 25, 2026
@olpipi olpipi requested review from praasz and t-jankowski March 25, 2026 16:25
@praasz praasz self-assigned this Mar 27, 2026
read_tensor_via_ifstream(file_name, *tensor.get(), offset_in_bytes);
return wrap_obj_to_viewtensor(tensor, tensor->data(), element_type, static_shape);
auto result = wrap_obj_to_viewtensor(tensor, tensor->data(), element_type, static_shape);
set_tensor_source_id(result, std::filesystem::hash_value(file_name));
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Check what is done for mmap obeject.

The id could be same for path only if offset is 0. for different offset it could be not correct.
This is some topic to sovle how creates these IDs in common way.

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: inference OpenVINO Runtime library - Inference

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants