Updated load_config mapping to make it a pass through for OV properties #782
+16
−94
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ankitm3k
force-pushed
the
ankit/load_config_v2
branch
from
07b08c0 to
3286d38
Compare
MayureshV1
approved these changes
Aug 22, 2025
There was a problem hiding this comment.
LGTM !
Testing has shown that this implementation matches the ABI-EP and load_config implementation is now passthrough to OpenVINO properties.
Most of the positive and negative tests have been validated.
Accuracy Mode, Cache_DIR, Cache_Mode, PERFORMANCE_HINT< EXECUTION_MODE_HINT all work.
Couple of OV bugs are on setting INFERENCE_PRECISION with incorrect values like FP16 still pass and LOG_LEVEL does nto work for devices like GPU.
MayureshV1
changed the title
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This PR simplifies the load_config mapping feature making the implementation deferring all the property handling at OV toolkit level & cleaner just like ABI EP.
Note: The onus is on user here on usage of valid properties while using load_config as its transparent at OVEP layer