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.

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

[Feature] Paf detector #651

Closed
wants to merge 100 commits into from
Closed

[Feature] Paf detector #651

wants to merge 100 commits into from

Conversation

luminxu
Copy link
Collaborator

@luminxu luminxu commented May 18, 2021

  1. Refactor bottom-up inference (disentangle heatmaps and tags).
  2. Add 'align_corners' in config.
  3. Add paf detector & head & loss & post-processing & demo

@luminxu luminxu requested a review from jin-s13 May 18, 2021 15:51
@jin-s13

This comment has been minimized.

@codecov
Copy link

codecov bot commented May 26, 2021

Codecov Report

Merging #651 (1e4e56a) into paf (c4f7c1f) will decrease coverage by 2.70%.
The diff coverage is 52.03%.

❗ Current head 1e4e56a differs from pull request most recent head 84671ab. Consider uploading reports for the commit 84671ab to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##              paf     #651      +/-   ##
==========================================
- Coverage   82.96%   80.25%   -2.71%     
==========================================
  Files         167      173       +6     
  Lines       12675    13418     +743     
  Branches     2060     2233     +173     
==========================================
+ Hits        10516    10769     +253     
- Misses       1634     2079     +445     
- Partials      525      570      +45     
Flag Coverage Δ
unittests 80.25% <52.03%> (-2.71%) ⬇️

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

Impacted Files Coverage Δ
mmpose/models/heads/topdown_heatmap_simple_head.py 84.84% <ø> (ø)
mmpose/models/heads/paf_head.py 14.54% <14.54%> (ø)
...pose/datasets/datasets/bottom_up/bottom_up_coco.py 73.11% <17.14%> (-12.96%) ⬇️
mmpose/models/detectors/paf.py 19.83% <19.83%> (ø)
mmpose/core/post_processing/group.py 41.20% <22.22%> (-41.43%) ⬇️
mmpose/models/losses/mse_loss.py 89.10% <28.57%> (-9.75%) ⬇️
mmpose/apis/inference.py 58.82% <33.33%> (-0.78%) ⬇️
...ets/datasets/bottom_up/bottom_up_coco_wholebody.py 82.08% <36.84%> (-7.57%) ⬇️
mmpose/models/heads/deconv_head.py 60.31% <60.31%> (ø)
mmpose/models/detectors/associative_embedding.py 74.59% <60.86%> (-4.31%) ⬇️
... and 68 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c4f7c1f...84671ab. Read the comment docs.

@jin-s13 jin-s13 changed the base branch from paf to master May 26, 2021 11:47
@jin-s13 jin-s13 changed the base branch from master to paf May 26, 2021 11:47
@jin-s13 jin-s13 changed the base branch from paf to master June 7, 2021 03:33
@jin-s13 jin-s13 changed the base branch from master to paf June 7, 2021 03:33
@jin-s13 jin-s13 changed the base branch from paf to master June 7, 2021 03:35
@jin-s13 jin-s13 changed the base branch from master to paf June 7, 2021 04:47
@CLAassistant
Copy link

CLAassistant commented Aug 3, 2021

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
2 out of 3 committers have signed the CLA.

✅ jin-s13
✅ luminxu
❌ 金晟


金晟 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.

@ly015 ly015 mentioned this pull request Aug 11, 2021
8 tasks
@jin-s13 jin-s13 changed the title Paf detector [Feature] Paf detector Sep 15, 2021
@jin-s13 jin-s13 mentioned this pull request Oct 13, 2021
@ly015 ly015 removed the status/WIP work in progress label Jan 26, 2022
@ly015 ly015 closed this Jan 26, 2022
@ly015 ly015 deleted the paf_detector branch September 2, 2022 03:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants