-
Notifications
You must be signed in to change notification settings - Fork 135
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
Enhance import performance for built-in plugins #1031
Enhance import performance for built-in plugins #1031
Conversation
Signed-off-by: Kim, Vinnam <vinnam.kim@intel.com>
Signed-off-by: Kim, Vinnam <vinnam.kim@intel.com>
Signed-off-by: Kim, Vinnam <vinnam.kim@intel.com>
Signed-off-by: Kim, Vinnam <vinnam.kim@intel.com>
Signed-off-by: Kim, Vinnam <vinnam.kim@intel.com>
Signed-off-by: Kim, Vinnam <vinnam.kim@intel.com>
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## develop #1031 +/- ##
===========================================
+ Coverage 78.74% 78.86% +0.11%
===========================================
Files 237 239 +2
Lines 27097 27178 +81
Branches 5407 5418 +11
===========================================
+ Hits 21338 21433 +95
+ Misses 4497 4480 -17
- Partials 1262 1265 +3
Flags with carried forward coverage won't be shown. Click here to find out more.
☔ View full report in Codecov by Sentry. |
Signed-off-by: Kim, Vinnam <vinnam.kim@intel.com>
d8f1fd0 added |
Signed-off-by: Kim, Vinnam <vinnam.kim@intel.com>
Signed-off-by: Kim, Vinnam <vinnam.kim@intel.com>
Signed-off-by: Kim, Vinnam <vinnam.kim@intel.com>
Summary
datumaro/components/lazy_plugin.py
,datumaro/plugins/specs.py
)DEFAULT_ENVIRONMENT
rather than creatingEnvironment()
for CLI codes.openvino-dev
to default extra deps and updatecontributing.md
datum
CLI latency by 1 sec (5 -> 4).(before)
(after)
How to test
I added a test for this change.
Checklist
License
Feel free to contact the maintainers if that's a concern.