From 4316a248aef4153e7922cdfe958cedbefca88d4f Mon Sep 17 00:00:00 2001 From: Attila Csok Date: Fri, 19 Apr 2024 05:51:39 -0700 Subject: [PATCH] [intel-npu] Unpublishing ov::internal properties (#23986) ### Details: Changes in intel-npu plugin: - Removing ov::internal properties from public supported_properties list - publishing model_priority property in supported_properties --- .../src/plugin/src/compiled_model.cpp | 14 +++++++------- .../intel_npu/src/plugin/src/plugin.cpp | 18 +++++++++--------- 2 files changed, 16 insertions(+), 16 deletions(-) diff --git a/src/plugins/intel_npu/src/plugin/src/compiled_model.cpp b/src/plugins/intel_npu/src/plugin/src/compiled_model.cpp index 8abb0f8b3d2..8b8be58e339 100644 --- a/src/plugins/intel_npu/src/plugin/src/compiled_model.cpp +++ b/src/plugins/intel_npu/src/plugin/src/compiled_model.cpp @@ -263,10 +263,16 @@ void CompiledModel::initialize_properties() { [](const Config& config) { return config.get(); }}}, + {ov::hint::model_priority.name(), + {true, + ov::PropertyMutability::RO, + [](const Config& config) { + return config.get(); + }}}, // OV Internals // ========= {ov::internal::supported_properties.name(), - {true, + {false, ov::PropertyMutability::RO, [&](const Config&) { static const std::vector supportedProperty{ @@ -276,12 +282,6 @@ void CompiledModel::initialize_properties() { }}}, // NPU Private // ========= - {ov::hint::model_priority.name(), - {false, - ov::PropertyMutability::RO, - [](const Config& config) { - return config.get(); - }}}, {ov::intel_npu::tiles.name(), {false, ov::PropertyMutability::RO, diff --git a/src/plugins/intel_npu/src/plugin/src/plugin.cpp b/src/plugins/intel_npu/src/plugin/src/plugin.cpp index 0e8b4b1543c..c22fdc4e944 100644 --- a/src/plugins/intel_npu/src/plugin/src/plugin.cpp +++ b/src/plugins/intel_npu/src/plugin/src/plugin.cpp @@ -291,22 +291,28 @@ Plugin::Plugin() const auto specifiedDeviceName = get_specified_device_name(config); return _metrics->GetFullDeviceName(specifiedDeviceName); }}}, + {ov::hint::model_priority.name(), + {true, + ov::PropertyMutability::RW, + [](const Config& config) { + return config.get(); + }}}, // OV Internals // ========= {ov::internal::caching_properties.name(), - {true, + {false, ov::PropertyMutability::RO, [&](const Config&) { return _metrics->GetCachingProperties(); }}}, {ov::internal::exclusive_async_requests.name(), - {true, + {false, ov::PropertyMutability::RW, [](const Config& config) { return config.get(); }}}, {ov::internal::supported_properties.name(), - {true, + {false, ov::PropertyMutability::RO, [&](const Config&) { return _metrics->GetInternalSupportedProperties(); @@ -333,12 +339,6 @@ Plugin::Plugin() }}}, // NPU Private // ========= - {ov::hint::model_priority.name(), - {false, - ov::PropertyMutability::RW, - [](const Config& config) { - return config.get(); - }}}, {ov::intel_npu::dma_engines.name(), {false, ov::PropertyMutability::RW,