mmpose icon indicating copy to clipboard operation
mmpose copied to clipboard

[Enhance] Reform multiviewpose

Open wusize opened this issue 2 years ago • 2 comments

Motivation

Reformulate "DetectAndRegress" so that it has similar structure with a bottom-up method

Modification

BC-breaking (Optional)

Use cases (Optional)

Checklist

Before PR:

  • [ ] I have read and followed the workflow indicated in the CONTRIBUTING.md to create this PR.
  • [ ] Pre-commit or linting tools indicated in CONTRIBUTING.md are used to fix the potential lint issues.
  • [ ] Bug fixes are covered by unit tests, the case that causes the bug should be added in the unit tests.
  • [ ] New functionalities are covered by complete unit tests. If not, please add more unit tests to ensure correctness.
  • [ ] The documentation has been modified accordingly, including docstring or example tutorials.

After PR:

  • [ ] CLA has been signed and all committers have signed the CLA in this PR.

wusize avatar Dec 02 '22 06:12 wusize

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


wusize seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

CLAassistant avatar Dec 02 '22 06:12 CLAassistant

Codecov Report

Base: 84.03% // Head: 83.98% // Decreases project coverage by -0.05% :warning:

Coverage data is based on head (07ead74) compared to base (1f11d6c). Patch coverage: 19.04% of modified lines in pull request are covered.

:exclamation: Current head 07ead74 differs from pull request most recent head 851d74a. Consider uploading reports for the commit 851d74a to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1853      +/-   ##
==========================================
- Coverage   84.03%   83.98%   -0.05%     
==========================================
  Files         241      241              
  Lines       20869    20879      +10     
  Branches     3609     3612       +3     
==========================================
- Hits        17537    17535       -2     
- Misses       2406     2414       +8     
- Partials      926      930       +4     
Flag Coverage Δ
unittests 83.91% <19.04%> (-0.05%) :arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
mmpose/models/detectors/multiview_pose.py 68.12% <15.00%> (-1.45%) :arrow_down:
mmpose/datasets/pipelines/top_down_transform.py 76.38% <100.00%> (ø)
mmpose/models/utils/misc.py 71.42% <0.00%> (-28.58%) :arrow_down:
mmpose/datasets/pipelines/shared_transform.py 81.53% <0.00%> (-0.91%) :arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

codecov[bot] avatar Jan 18 '23 13:01 codecov[bot]