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

[intel-npu] Adding new read-only property: NPU_COMPILER_VERSION #28314

Merged
merged 4 commits into from
Jan 13, 2025
Merged

[intel-npu] Adding new read-only property: NPU_COMPILER_VERSION #28314

merged 4 commits into from
Jan 13, 2025

Conversation

csoka
Copy link
Contributor

@csoka csoka commented Jan 8, 2025 •
edited
Loading

Details:

  • extending compiler adapter interfaces with getApiVersion functions
  • adding new public read-only property to expose compiler api version

Tickets:

  • EISW-151484

@csoka csoka requested review from a team as code owners January 8, 2025 12:12
@csoka csoka requested review from tsavina and removed request for a team January 8, 2025 12:12
@github-actions github-actions bot added category: inference OpenVINO Runtime library - Inference category: docs OpenVINO documentation category: CPP API OpenVINO CPP API bindings category: NPU OpenVINO NPU plugin labels Jan 8, 2025
@csoka csoka requested a review from lmielick January 8, 2025 12:13
ilya-lavrenov
ilya-lavrenov reviewed Jan 8, 2025
View reviewed changes
src/inference/include/openvino/runtime/intel_npu/properties.hpp Outdated
* Read-only property to get NPU compiler API version
* @ingroup ov_runtime_npu_prop_cpp_api
*/
static constexpr ov::Property<uint32_t, ov::PropertyMutability::RO> compiler_api_version{"NPU_COMPILER_API_VERSION"};
Copy link
Contributor

Choose a reason for hiding this comment

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

please, add python api

@lmielick lmielick changed the title [intel-npu] Adding new read-only property: NPU_COMPILER_API_VERSION [intel-npu] Adding new read-only property: NPU_COMPILER_VERSION Jan 9, 2025
@csoka csoka requested a review from a team as a code owner January 9, 2025 22:34
@github-actions github-actions bot added the category: Python API OpenVINO Python bindings label Jan 9, 2025
@csoka csoka requested a review from akuporos January 10, 2025 12:33
@PatrikStepan PatrikStepan added this pull request to the merge queue Jan 13, 2025
Merged via the queue into openvinotoolkit:master with commit 2b0f127 Jan 13, 2025
186 checks passed
MirceaDan99 pushed a commit to MirceaDan99/openvino that referenced this pull request Jan 22, 2025
@csoka @MirceaDan99
[intel-npu] Adding new read-only property: NPU_COMPILER_VERSION (open…
63e97a2 
…vinotoolkit#28314)

### Details:
 - extending compiler adapter interfaces with getApiVersion functions
 - adding new public read-only property to expose compiler api version

### Tickets:
 - *EISW-151484*
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 left review comments

@lmielick lmielick lmielick approved these changes

@akuporos akuporos akuporos approved these changes

@PatrikStepan PatrikStepan PatrikStepan approved these changes

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

Assignees
No one assigned
Labels
category: CPP API OpenVINO CPP API bindings category: docs OpenVINO documentation category: inference OpenVINO Runtime library - Inference category: NPU OpenVINO NPU plugin category: Python API OpenVINO Python bindings
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@csoka @ilya-lavrenov @PatrikStepan @akuporos @lmielick