Skip to content

[core] Remove and change deprecated data() member in tensor #32569

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

[core] Remove and change deprecated data() member in tensor #32569

wants to merge 5 commits into from
+44 −64

Conversation

@praasz
Copy link
Contributor

@praasz praasz commented Oct 27, 2025 •
edited
Loading

@praasz
Remove and change deprecated data() member in tensor
a8dc8ab 
Signed-off-by: Raasz, Pawel <pawel.raasz@intel.com>
@praasz praasz added this to the 2026.0 milestone Oct 27, 2025
@praasz praasz added the WIP work in progress label Oct 27, 2025
@praasz praasz requested review from a team as code owners October 27, 2025 13:53
@praasz praasz requested review from a team as code owners October 27, 2025 13:53
@praasz praasz requested review from mlukasze and removed request for a team October 27, 2025 13:53
@github-actions github-actions bot added category: Core OpenVINO Core (aka ngraph) category: IE Tests OpenVINO Test: plugins and common category: GPU OpenVINO GPU plugin category: CPU OpenVINO CPU plugin category: HETERO OpenVINO HETERO plugin category: samples OpenVINO Runtime Samples category: TEMPLATE OpenVINO Template plugin category: CPP API OpenVINO CPP API bindings category: PROXY OpenVINO proxy plugin category: NPU OpenVINO NPU plugin category: NPUW NPUW plugin labels Oct 27, 2025
@Wovchena
Copy link
Contributor

Wovchena commented Oct 27, 2025

GenAI needs to be fixed first. Will you do that (or maybe already started)?

@praasz
Copy link
Contributor Author

praasz commented Oct 28, 2025

GenAI needs to be fixed first. Will you do that (or maybe already started)?

Will fix that, it looks also in NVIDIA plugin must be fixed also.

dmatveev
dmatveev approved these changes Oct 28, 2025
View reviewed changes
Copy link
Contributor

@dmatveev dmatveev left a comment

Choose a reason for hiding this comment

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

@smirnov-alexey please have a look on NPUW-related changes (I had mine, don't see a problem here)

@praasz praasz mentioned this pull request Oct 28, 2025
Merged
3 tasks
@github-actions github-actions bot removed category: GPU OpenVINO GPU plugin category: HETERO OpenVINO HETERO plugin category: TEMPLATE OpenVINO Template plugin category: PROXY OpenVINO proxy plugin labels Oct 30, 2025
github-merge-queue bot pushed a commit to openvinotoolkit/openvino.genai that referenced this pull request Nov 6, 2025
@praasz
Correct usage of ov::Tensor::data for 26.0 release (#2934)
d1e95e4 
## Description
- Update usage of ov::Tensor::data() member for OV 26.0 to fix build
issues.
- No test update, functionality not changed.
- No docs update required.

Blocked PR:
- openvinotoolkit/openvino#32569

<!--- Jira ticket number (e.g., 123). Delete if there's no ticket. Don't
include full link or project name. -->
Ticket: 
CVS-174872

## Checklist:
- [x] Tests have been updated or added to cover the new code <!--- If
the change isn't maintenance related, update the tests at
https://github.com/openvinotoolkit/openvino.genai/tree/master/tests or
explain in the description why the tests don't need an update. -->
- [x] This patch fully addresses the ticket. <!--- If follow-up pull
requests are needed, specify in description. -->
- [x] I have made corresponding changes to the documentation

Signed-off-by: Raasz, Pawel <pawel.raasz@intel.com>
@praasz praasz removed WIP work in progress do_not_review labels Nov 6, 2025
@praasz praasz mentioned this pull request Nov 6, 2025
Merged
3 tasks
Wovchena pushed a commit to openvinotoolkit/openvino.genai that referenced this pull request Nov 6, 2025
@praasz
Update use tensor data member (#2978)
fee79fe 
## Description
- Remove dead code which cause compilation error with updated
ov::Tensor::data() member. for OV 26.0 to fix build issues.
- No test update, functionality not changed.
- No docs update required.

Blocked PR:
- openvinotoolkit/openvino#32569

<!--- Jira ticket number (e.g., 123). Delete if there's no ticket. Don't
include full link or project name. -->
Ticket: 
CVS-174872

## Checklist:
- [x] Tests have been updated or added to cover the new code <!--- If
the change isn't maintenance related, update the tests at
https://github.com/openvinotoolkit/openvino.genai/tree/master/tests or
explain in the description why the tests don't need an update. -->
- [x] This patch fully addresses the ticket. <!--- If follow-up pull
requests are needed, specify in description. -->
- [x] I have made corresponding changes to the documentation

---------

Signed-off-by: Raasz, Pawel <pawel.raasz@intel.com>
@mlukasze mlukasze enabled auto-merge November 7, 2025 06:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dmatveev dmatveev dmatveev approved these changes

@Wovchena Wovchena Wovchena approved these changes

@smirnov-alexey smirnov-alexey smirnov-alexey approved these changes

@t-jankowski t-jankowski t-jankowski approved these changes

@CuriousPanCake CuriousPanCake CuriousPanCake approved these changes

@mlukasze mlukasze Awaiting requested review from mlukasze mlukasze is a code owner automatically assigned from openvinotoolkit/openvino-maintainers

Assignees

No one assigned

Labels

category: Core OpenVINO Core (aka ngraph) category: CPP API OpenVINO CPP API bindings category: CPU OpenVINO CPU plugin category: IE Tests OpenVINO Test: plugins and common category: NPU OpenVINO NPU plugin category: NPUW NPUW plugin category: samples OpenVINO Runtime Samples

Projects

None yet

Milestone

2026.0

Development

Successfully merging this pull request may close these issues.

7 participants

@praasz @Wovchena @dmatveev @smirnov-alexey @t-jankowski @CuriousPanCake @mlukasze