Skip to content

[core, GPU, NPU]Remove deprecated code in core for ov::cache_dir #32658

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 4 commits into
base: master
Choose a base branch
from
Open

[core, GPU, NPU]Remove deprecated code in core for ov::cache_dir #32658

wants to merge 4 commits into from
+59 −130

Conversation

@praasz
Copy link
Contributor

@praasz praasz commented Nov 3, 2025 •
edited
Loading

Details:

  • Remove deprecate code in core to forward ov::cache_dir to hardware plugins
  • Remove in GPU,NPU ov::cache_dir from supported properties
  • Update tests

Tickets:

@praasz
Remove pass ov::cache_dir to HW devices
cdac795 
- remove ov::cache_dir as supported property in GPU and NPU
- update core tests

Signed-off-by: Pawel Raasz <[email protected]>
@praasz praasz added this to the 2026.0 milestone Nov 3, 2025
@praasz praasz requested a review from a team as a code owner November 3, 2025 14:08
@praasz praasz requested review from a team as code owners November 3, 2025 14:08
@praasz praasz added WIP work in progress do_not_merge labels Nov 3, 2025
@github-actions github-actions bot added category: inference OpenVINO Runtime library - Inference category: GPU OpenVINO GPU plugin category: NPU OpenVINO NPU plugin labels Nov 3, 2025
@praasz
Update Node JS tests
e9727af 
Signed-off-by: Pawel Raasz <[email protected]>
@praasz praasz requested a review from a team as a code owner November 3, 2025 18:04
@github-actions github-actions bot added the category: JS API OpenVino JS API Bindings label Nov 3, 2025
Retribution98
Retribution98 reviewed Nov 4, 2025
View reviewed changes
@praasz praasz removed the WIP work in progress label Nov 4, 2025
praasz added 2 commits November 4, 2025 15:08
@praasz
Update JS unsupported property test
feef71a 
Signed-off-by: Pawel Raasz <[email protected]>
@praasz
Fix style
75154a6 
Signed-off-by: Pawel Raasz <[email protected]>
@praasz praasz requested a review from Retribution98 November 5, 2025 11:45
Retribution98
Retribution98 approved these changes Nov 5, 2025
View reviewed changes
Copy link
Contributor

@Retribution98 Retribution98 left a comment

Choose a reason for hiding this comment

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

JS part is ok

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

Reviewers

@Retribution98 Retribution98 Retribution98 approved these changes

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

@PatrikStepan PatrikStepan Awaiting requested review from PatrikStepan

@isanghao isanghao Awaiting requested review from isanghao

Assignees

@PatrikStepan PatrikStepan

@isanghao isanghao

@olpipi olpipi

@barnasm1 barnasm1

Labels

category: GPU OpenVINO GPU plugin category: inference OpenVINO Runtime library - Inference category: JS API OpenVino JS API Bindings category: NPU OpenVINO NPU plugin

Projects

None yet

Milestone

2026.0

Development

Successfully merging this pull request may close these issues.

7 participants

@praasz @Retribution98 @t-jankowski @PatrikStepan @isanghao @olpipi @barnasm1