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

[GPU] POC for shared plugin config #28667

Open
wants to merge 26 commits into
base: master
Choose a base branch
from
Open

[GPU] POC for shared plugin config #28667

wants to merge 26 commits into from
+2,023 −1,909

Conversation

vladimir-paramuzov
Copy link
Contributor

Details:

  • Added common base class for plugin configuration and migration in GPU plugin.
  • New config defines options as typed members of config class which shall provide more efficient access to the value than AnyMap
  • Property setting is possible before finalization only which shall localize the point when config become immutable.
  • All config options may be read from environment as well as from json config file when OV is built w/ debug caps.
  • Debug and release options are defined and handled similarly now

@vladimir-paramuzov
Base impl
b8b8ff2 
Signed-off-by: Vladimir Paramuzov <[email protected]>
@vladimir-paramuzov
make it common
344e277 
Signed-off-by: Vladimir Paramuzov <[email protected]>
@vladimir-paramuzov
env and config
d7f73b3 
Signed-off-by: Vladimir Paramuzov <[email protected]>
@vladimir-paramuzov
Replace old config & fixes
5613eb8 
Signed-off-by: Vladimir Paramuzov <[email protected]>
@vladimir-paramuzov
prefix for config members and unit tests
d70d23b 
Signed-off-by: Vladimir Paramuzov <[email protected]>
@vladimir-paramuzov
added visibility for options
2b63c30 
Signed-off-by: Vladimir Paramuzov <[email protected]>
@vladimir-paramuzov
remove old config
cec123b 
Signed-off-by: Vladimir Paramuzov <[email protected]>
@vladimir-paramuzov
enhancements
a7fda69 
Signed-off-by: Vladimir Paramuzov <[email protected]>
@vladimir-paramuzov
update behavior for set/get property. Add help message
45ba524 
Signed-off-by: Vladimir Paramuzov <[email protected]>
@vladimir-paramuzov
refactor
af121ec 
Signed-off-by: Vladimir Paramuzov <[email protected]>
@vladimir-paramuzov
Hide config class members
4b90a8a 
Signed-off-by: Vladimir Paramuzov <[email protected]>
@vladimir-paramuzov
Options visibility update
5a2a5d5 
Signed-off-by: Vladimir Paramuzov <[email protected]>
@vladimir-paramuzov
Fixes and visit_attributes method impl
23deb9d 
Signed-off-by: Vladimir Paramuzov <[email protected]>
@vladimir-paramuzov
Refactor debug knobs
25c723b 
Signed-off-by: Vladimir Paramuzov <[email protected]>
@vladimir-paramuzov
split set_prop and set_user_prop again
452d2ca 
Signed-off-by: Vladimir Paramuzov <[email protected]>
@vladimir-paramuzov
extended bool any parsing options
6c013b0 
Signed-off-by: Vladimir Paramuzov <[email protected]>
@vladimir-paramuzov
debug properties wip
3bd344d 
Signed-off-by: Vladimir Paramuzov <[email protected]>
@vladimir-paramuzov
fix apply rt info
c238064 
Signed-off-by: Vladimir Paramuzov <[email protected]>
@vladimir-paramuzov
wip
81c8509 
Signed-off-by: Vladimir Paramuzov <[email protected]>
@vladimir-paramuzov
[GPU] Global static vars. Removed old debug config
8f9d035 
Signed-off-by: Vladimir Paramuzov <[email protected]>
@vladimir-paramuzov
fix visitors
c6637f0 
Signed-off-by: Vladimir Paramuzov <[email protected]>
@vladimir-paramuzov
build fixes
a025842 
Signed-off-by: Vladimir Paramuzov <[email protected]>
@vladimir-paramuzov
minor fixes
2c991e8 
Signed-off-by: Vladimir Paramuzov <[email protected]>
@vladimir-paramuzov
cut off debug properties for release build
34776f5 
Signed-off-by: Vladimir Paramuzov <[email protected]>
@vladimir-paramuzov
config clone. Visibility fixes
ac836c9 
Signed-off-by: Vladimir Paramuzov <[email protected]>
@vladimir-paramuzov
added comment about config copy
73cf4ac 
Signed-off-by: Vladimir Paramuzov <[email protected]>
@vladimir-paramuzov vladimir-paramuzov requested review from a team as code owners January 24, 2025 13:09
@vladimir-paramuzov vladimir-paramuzov requested review from a team as code owners January 24, 2025 13:09
@github-actions github-actions bot added category: inference OpenVINO Runtime library - Inference category: Core OpenVINO Core (aka ngraph) category: GPU OpenVINO GPU plugin category: CPU OpenVINO CPU plugin category: build OpenVINO cmake script / infra labels Jan 24, 2025
@dmitry-gorokhov dmitry-gorokhov mentioned this pull request Jan 24, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
category: build OpenVINO cmake script / infra category: Core OpenVINO Core (aka ngraph) category: CPU OpenVINO CPU plugin category: GPU OpenVINO GPU plugin category: inference OpenVINO Runtime library - Inference
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@vladimir-paramuzov