-
Notifications
You must be signed in to change notification settings - Fork 633
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
[Fix] Fixed device_id in tools/test.py for the CPU. #58
[Fix] Fixed device_id in tools/test.py for the CPU. #58
Conversation
Codecov Report
@@ Coverage Diff @@
## master #58 +/- ##
==========================================
+ Coverage 67.46% 67.60% +0.13%
==========================================
Files 173 173
Lines 5733 5735 +2
Branches 910 910
==========================================
+ Hits 3868 3877 +9
+ Misses 1583 1578 -5
+ Partials 282 280 -2
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
* fix bugs for mmcls performance test * fix yapf * add comments of CLASSES attribute
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.
LGTM
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.
LGTM
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.
LGTM
Motivation
It was found that
tools/test.py
with argument--device cpu
crashes with the following error:This PR proposes a fix for this problem.
Modification
mmdeploy/backend/openvino/wrapper.py
: Fixed work with the dictionary for Python 3.8.mmdeploy/utils/device.py
: The documentation of theparse_device_id
function was updated.tests/test_utils/test_util.py
: Added tests for functionparse_device_id
.tools/test.py
: Added fix forcpu
.