Skip to content

Conversation

@razvanapetroaie
Copy link
Contributor

@razvanapetroaie razvanapetroaie commented Nov 18, 2025

Details:

  • The current boolean option that switches between the two implementations will be discarded in favor of this newer enum-based one. This type of data will allow us to patch the current implementation and handle compatiblity cases in a custom manner.
  • To facilitate migration, only the option supported by the compiler will be registered in the config object (if set explicitly). So the flow is: check the value used by use_base_model_serializer (this one has priority) or model_serializer_version -> store the value and remove the options before building the config -> write in the config only the option supported by the compiler.

Tickets:

@razvanapetroaie razvanapetroaie requested review from a team as code owners November 18, 2025 18:24
@github-actions github-actions bot added the category: NPU OpenVINO NPU plugin label Nov 18, 2025
@razvanapetroaie razvanapetroaie marked this pull request as draft November 18, 2025 18:27
@razvanapetroaie razvanapetroaie force-pushed the CVS-173711-add-version-option branch from 6b71aab to 59630db Compare November 18, 2025 19:05
@razvanapetroaie razvanapetroaie marked this pull request as ready for review November 18, 2025 19:08
@razvanapetroaie razvanapetroaie marked this pull request as draft November 18, 2025 19:09
@razvanapetroaie razvanapetroaie added this pull request to the merge queue Nov 20, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Nov 20, 2025
@StefaniaHergane StefaniaHergane added this pull request to the merge queue Nov 20, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Nov 20, 2025
@StefaniaHergane StefaniaHergane added this pull request to the merge queue Nov 20, 2025
Merged via the queue into openvinotoolkit:master with commit 275f743 Nov 20, 2025
177 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category: NPU OpenVINO NPU plugin

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants