-
Notifications
You must be signed in to change notification settings - Fork 1.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
Support COCO-Wholebody #133
Conversation
Codecov Report
@@ Coverage Diff @@
## master #133 +/- ##
==========================================
- Coverage 82.26% 81.49% -0.78%
==========================================
Files 84 85 +1
Lines 4871 4976 +105
Branches 857 870 +13
==========================================
+ Hits 4007 4055 +48
- Misses 713 767 +54
- Partials 151 154 +3
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
mmpose/datasets/datasets/top_down/topdown_coco_wholebody_dataset.py
Outdated
Show resolved
Hide resolved
|
||
return rec | ||
|
||
def _coco_keypoint_results_one_category_kernel(self, data_pack): |
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.
looks like the test coverage of this function is low
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.
Maybe add the 'unit-test for dataset testing' in a separate PR.
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.
fine
|
||
return cat_results | ||
|
||
def _do_python_keypoint_eval(self, res_file): |
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.
and this one
mmpose/datasets/datasets/top_down/topdown_coco_wholebody_dataset.py
Outdated
Show resolved
Hide resolved
need a pr to update repo's readme, and the change log |
No description provided.