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.

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

[GPU] Set cache encryption callbacks as write-only and fix is_mutable() #28808

Merged
merged 3 commits into from
Feb 10, 2025
Merged

[GPU] Set cache encryption callbacks as write-only and fix is_mutable() #28808

merged 3 commits into from
Feb 10, 2025

Conversation

tkrupa-intel
Copy link
Contributor

@tkrupa-intel tkrupa-intel commented Feb 4, 2025 •
edited by ilya-lavrenov
Loading

Details:

  • cache encryption callbacks changed to write-only in GPU Plugin
  • is_mutable() now also true for write-only properties
  • cache encryption behavior is not changed

Tickets:

@tkrupa-intel tkrupa-intel requested review from a team as code owners February 4, 2025 09:34
@github-actions github-actions bot added the category: GPU OpenVINO GPU plugin label Feb 4, 2025
@ilya-lavrenov ilya-lavrenov added this to the 2025.1 milestone Feb 4, 2025
@tkrupa-intel tkrupa-intel requested a review from a team as a code owner February 6, 2025 10:40
@github-actions github-actions bot added category: inference OpenVINO Runtime library - Inference category: CPP API OpenVINO CPP API bindings labels Feb 6, 2025
@tkrupa-intel tkrupa-intel changed the title [GPU] Remove cache encryption callbacks from supported properties exposed to user [GPU] Set cache encryption callbacks as write-only and fix is_mutable() Feb 6, 2025
@ilya-lavrenov ilya-lavrenov self-assigned this Feb 6, 2025
@p-durandin p-durandin added this pull request to the merge queue Feb 10, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 10, 2025
@p-durandin p-durandin added this pull request to the merge queue Feb 10, 2025
@tkrupa-intel tkrupa-intel mentioned this pull request Feb 10, 2025
Merged
Merged via the queue into openvinotoolkit:master with commit fa3a158 Feb 10, 2025
185 checks passed
github-merge-queue bot pushed a commit that referenced this pull request Feb 11, 2025
@tkrupa-intel
Python API: is_mutable() fix (#28907)
d00cc18 
### Details:
 - followup to #28808
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ilya-lavrenov ilya-lavrenov ilya-lavrenov approved these changes

@akuporos akuporos akuporos left review comments

@helena-intel helena-intel helena-intel left review comments

@praasz praasz praasz approved these changes

Assignees

@ilya-lavrenov ilya-lavrenov

@praasz praasz

Labels
category: CPP API OpenVINO CPP API bindings category: GPU OpenVINO GPU plugin category: inference OpenVINO Runtime library - Inference
Projects
None yet
Milestone
2025.1
Development

Successfully merging this pull request may close these issues.
6 participants
@tkrupa-intel @ilya-lavrenov @akuporos @helena-intel @praasz @p-durandin