-
Notifications
You must be signed in to change notification settings - Fork 3k
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.
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
OpenVINO-EP v4.0 Release PR with OpenVINO 2022.1 #11025
Merged
Merged
Changes from all commits
Commits
Show all changes
101 commits
Select commit
Hold shift + click to select a range
133fb01
Enabling ov-ep for 2022.1 Release
MaajidKhan 948941f
Fix for output mismatch b/w OpenVINO and ONNX
MaajidKhan 140c599
Enabling Adobe ops
MaajidKhan d5908a6
Removing irrelevant conditions
MaajidKhan 72cc91b
Enable upsample op
MaajidKhan 4b55164
Enable Adobe proxy-e model
MaajidKhan 155765b
Removing any extra conditions for Opset13 ops
MaajidKhan 8455317
Opset13 changes
MaajidKhan 7ad2a02
Exception handling for devices
hdgx 47116d2
Added comments
hdgx 34a9c7a
Implement GPU Throttling feature
MaajidKhan dc24bad
Renaming the runtime config option
MaajidKhan 117e3e9
Added the user to video and users group
mayavijx 9cdbdf3
Handling_GPU.0_GPU.1
hdgx 2c94fc3
Handling special conditions
MaajidKhan e4a455b
Modification to include new api 2.0 changes in the code
MaajidKhan a14b8c0
Added opset13 changes
MaajidKhan e264ca9
Enabling ov-ep for 2022.1 Release
MaajidKhan 049d8c9
Fix for output mismatch b/w OpenVINO and ONNX
MaajidKhan e0cc70f
Enabling Adobe ops
MaajidKhan d0141be
Removing irrelevant conditions
MaajidKhan ec96986
Enable upsample op
MaajidKhan b9e87d6
Enable Adobe proxy-e model
MaajidKhan ca70463
Removing any extra conditions for Opset13 ops
MaajidKhan fde21af
Opset13 changes
MaajidKhan c9f7ea6
Exception handling for devices
hdgx d95f8df
Added comments
hdgx 1f663ba
Implement GPU Throttling feature
MaajidKhan 682c466
Renaming the runtime config option
MaajidKhan d5c3a16
Added the user to video and users group
mayavijx 90c2307
Handling_GPU.0_GPU.1
hdgx dcebe0a
Handling special conditions
MaajidKhan 94eae53
Added opset13 changes
MaajidKhan dfca9a5
Log comments updated
MaajidKhan cf8dbf3
Changes to enable 2.0 api
MaajidKhan 9e69fbc
Merge remote-tracking branch 'origin/enable_2022.1_branch' into sahar…
MaajidKhan 335ba34
Enabling ov-ep for 2022.1 Release
MaajidKhan 1e95f5c
Fix for output mismatch b/w OpenVINO and ONNX
MaajidKhan 0cdbe24
Enabling Adobe ops
MaajidKhan 6c1cbf3
Removing irrelevant conditions
MaajidKhan fff2c74
Enable upsample op
MaajidKhan 01f229c
Enable Adobe proxy-e model
MaajidKhan 2e1cbe5
Removing any extra conditions for Opset13 ops
MaajidKhan b819b34
Opset13 changes
MaajidKhan c4c0f39
Exception handling for devices
hdgx 23b51c8
Added comments
hdgx 59cf128
Implement GPU Throttling feature
MaajidKhan 4eb5a08
Renaming the runtime config option
MaajidKhan 2e87c12
Added the user to video and users group
mayavijx e2e88e3
Handling_GPU.0_GPU.1
hdgx 067e3dc
Handling special conditions
MaajidKhan d83517f
Added opset13 changes
MaajidKhan 7a9b39b
Merge remote-tracking branch 'origin/enable_2022.1_branch' into sahar…
MaajidKhan 30696d0
Merge branch 'enable_2022.1_branch' of https://github.com/intel/onnxr…
MaajidKhan ebdcb21
Merge branch 'enable_2022.1_branch' into sahar/2.0_api
MaajidKhan 1ca62ca
Merge pull request #141 from intel/sahar/2.0_api
sfatimar 9841ba9
Fix build issue
MaajidKhan e67d02b
Merge remote-tracking branch 'msft/master' into enable_2022.1_mar2
MaajidKhan 9578cef
Fixes issues
MaajidKhan a231d35
commit to make openvino_2021.4 compatible
MaajidKhan 2ea9e49
Fixed IO Buffer Optimization
MaajidKhan adb69e4
Fix output names issue
MaajidKhan 035d11c
Merge branch 'master' into enable_2022.1_branch
MaajidKhan 9aeaae4
Fix 2021.3 branch
MaajidKhan d30e522
Merge pull request #146 from intel/sahar/fix_2021.3
mohsinmx e6ef10b
Bug Fix for Multiple inputs/outputs
MaajidKhan 05fdbab
Add comments for the changes made
MaajidKhan 28a8d08
Merge pull request #147 from intel/multiple_inps_outs_bug_fix
sfatimar 7361b90
IO Buffer Changes
MaajidKhan be5de99
Merge branch 'enable_2022.1_branch' into sahar/io_buffer_20
MaajidKhan 459aaec
Commit for Disabling GPU Throttling for 2021.4
MaajidKhan c25a2ec
Merge pull request #151 from intel/sahar/io_buffer_20
sfatimar 3c21187
Updated branch
MaajidKhan 20a492b
Merge branch 'enable_2022.1_branch' of https://github.com/intel/onnxr…
MaajidKhan 0c6a86f
Fix windows build
MaajidKhan e10d979
Fixed CPP Unit tests for CPU
MaajidKhan d5ffcb4
Fixed first set of GPU Tests
MaajidKhan f250a84
Fixed additional failing tests on GPU
MaajidKhan 7c763a0
Added Expand op support for CPU
MaajidKhan b341aee
Added condition for squeeze op
MaajidKhan 3af3377
Add support for LessOrEqual op function
MaajidKhan 1c41635
OV Interface wait for replaced by indefinite wait call
MaajidKhan 72ebda5
Merge branch 'master' into enable_2022.1_branch
MaajidKhan d29123b
use names from ONNX model to access OV tensors
MaajidKhan 8cd78ad
Fixes Myriad unit tests and other issues
MaajidKhan dc247de
Fix segfault issue
MaajidKhan eb54b2d
Fixed build isuse with ov 2021.4 with I/O buffer
MaajidKhan 8c00cb6
Disables performance counters for I/O Buffer
MaajidKhan 4f47954
Fixed inputs/outputs mismatch for HDDL with 2022.1
amiraqx 2e87293
Fix to enable GPU FP16
MaajidKhan 63cfc77
Merge branch 'master' into enable_2022.1_branch
MaajidKhan 01f67c4
Enabled mlperf_ssd_mobilenet_300 model fully on CPU
MaajidKhan 4151eff
Added ov version specific dll packaging for nuget
mayavijx d9880cb
Fixed conditions for few ops
MaajidKhan 08f9381
Dockerfile updates
mayavijx 499a331
Merge branch 'master' into enable_2022.1_branch
MaajidKhan 92b83b3
Updated License Info
MaajidKhan 1fc1620
Merge branch 'master' into enable_2022.1_mar28
MaajidKhan 3b35966
Disabling mlperf_ssd_mobilenet_300 model
MaajidKhan 2561683
Disabling failing python CPU Tests
MaajidKhan e3e5958
Fixed flake8 python errors
MaajidKhan File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains 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
This file contains 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
This file contains 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
This file contains 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
This file contains 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
This file contains 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
This file contains 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
This file contains 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
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
using c api, we're supposed to maintain backwards compatibility (with abi)
i.e. a user can drop in a new version of onnxruntime shared lib (built with openvino ep)
and it would work. But here, applications built using earlier ort version would be using a version of OrtOpenVINOProviderOptions struct that did not contain this new field.
do you know if openvino ep users rely on abi? or will they always recompile their application with latest ort headers/library.
This is why we have moved towards using opaque structs to avoid this issue:
see #7808 as example.
we create a V2 version of struct, which the user does not directly manipulate. they use api's to construct and pass the struct in as option. the api's use key/value strings for EP options.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi George. As of today, all our users don't rely on abi. They usually recompile their application with latest ort headers/library with the version of OpenVINO they want to work with. However, I think moving forward, we would also like to move to use the opaque structs as mentioned in the above comments to have the backward compatibility support.