Skip to content

Upgrade OpenVINO Toolkit v2024.4#472

Merged
vthaniel merged 1 commit intoovep-develop-lnl-1.2from
jatin/upgrade_ovep_2024.4
Oct 13, 2024
Merged

Upgrade OpenVINO Toolkit v2024.4#472
vthaniel merged 1 commit intoovep-develop-lnl-1.2from
jatin/upgrade_ovep_2024.4

Conversation

@jatinwadhwa921
Copy link

Upgrade OpenVINO Toolkit v2024.4

@jatinwadhwa921 jatinwadhwa921 force-pushed the jatin/upgrade_ovep_2024.4 branch from c9f7427 to 93c70a9 Compare October 11, 2024 12:13
data_ops_ = new DataOps(graph_viewer_, V_2024_0, device_type_, npu_qdq_optimizer_enabled);
#elif OPENVINO_VERSION_MAJOR == 2024 && OPENVINO_VERSION_MINOR == 1
#if OPENVINO_VERSION_MAJOR == 2024 && OPENVINO_VERSION_MINOR == 1
data_ops_ = new DataOps(graph_viewer_, V_2024_1, device_type_, npu_qdq_optimizer_enabled);

Choose a reason for hiding this comment

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

We can remove 2024.1 check as we only commit for latest three releases of OV

Copy link
Author

Choose a reason for hiding this comment

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

fixed

Choose a reason for hiding this comment

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

I think you have removed 2024.0 (which should have been removed in last release itself)
We need to remove 2024.1 also right?

Copy link

@vthaniel vthaniel left a comment

Choose a reason for hiding this comment

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

CPP Unit tests on CPU are passing

@vthaniel vthaniel merged commit 6089543 into ovep-develop-lnl-1.2 Oct 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants