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

[Fix] Fix the incorrect labels for training vis_head with combined datasets #2550

Merged
merged 8 commits into from
Jul 27, 2023

Conversation

Ben-Louis
Copy link
Collaborator

Motivation

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.

@codecov
Copy link

codecov bot commented Jul 19, 2023

Codecov Report

Patch coverage: 68.75% and project coverage change: -0.05% ⚠️

Comparison is base (5e862cc) 80.83% compared to head (85d16ea) 80.79%.
Report is 19 commits behind head on dev-1.x.

❗ Current head 85d16ea differs from pull request most recent head 6145d3f. Consider uploading reports for the commit 6145d3f to get more accurate results

Additional details and impacted files
@@             Coverage Diff             @@
##           dev-1.x    #2550      +/-   ##
===========================================
- Coverage    80.83%   80.79%   -0.05%     
===========================================
  Files          230      230              
  Lines        14403    14469      +66     
  Branches      2489     2507      +18     
===========================================
+ Hits         11643    11690      +47     
- Misses        2122     2131       +9     
- Partials       638      648      +10     
Flag Coverage Δ
unittests 80.79% <68.75%> (-0.05%) ⬇️

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

Files Changed Coverage Δ
mmpose/datasets/transforms/common_transforms.py 84.25% <0.00%> (-0.28%) ⬇️
mmpose/datasets/transforms/formatting.py 79.76% <ø> (+1.85%) ⬆️
...s/heads/regression_heads/motion_regression_head.py 28.78% <11.11%> (-2.25%) ⬇️
mmpose/codecs/motionbert_label.py 75.55% <28.57%> (-2.86%) ⬇️
mmpose/models/pose_estimators/pose_lifter.py 37.28% <37.50%> (-0.55%) ⬇️
mmpose/models/pose_estimators/bottomup.py 70.00% <50.00%> (-0.69%) ⬇️
mmpose/models/pose_estimators/topdown.py 67.79% <50.00%> (-0.63%) ⬇️
mmpose/visualization/local_visualizer.py 63.40% <60.00%> (+1.09%) ⬆️
mmpose/apis/inferencers/pose3d_inferencer.py 82.72% <66.66%> (+0.09%) ⬆️
mmpose/models/heads/hybrid_heads/vis_head.py 83.15% <70.83%> (-4.94%) ⬇️
... and 9 more

... and 2 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Tau-J
Copy link
Collaborator

Tau-J commented Jul 19, 2023

Have you checked the influence on browse_dataset.py ?

@Ben-Louis
Copy link
Collaborator Author

Have you checked the influence on browse_dataset.py ?

Yes, I tried to browse the combined dataset body7, and the results seem reasonable

@Tau-J Tau-J merged commit abe09d3 into open-mmlab:dev-1.x Jul 27, 2023
Tau-J added a commit that referenced this pull request Oct 12, 2023
* update

* [Fix] Fix HRFormer log link

* [Feature] Add Application 'Just dance' (#2528)

* [Docs] Add advanced tutorial of implement new model. (#2539)

* [Doc] Update img (#2541)

* [Feature] Support MotionBERT (#2482)

* [Fix] Fix demo scripts (#2542)

* [Fix] Fix Pose3dInferencer keypoint shape bug (#2543)

* [Enhance] Add notifications when saving visualization results (#2545)

* [Fix] MotionBERT training and flip-test (#2548)

* [Docs] Enhance docs (#2555)

* [Docs] Fix links in doc (#2557)

* [Docs] add details (#2558)

* [Refactor] 3d human pose demo (#2554)

* [Docs] Update MotionBERT docs (#2559)

* [Refactor] Update the arguments of 3d inferencer to align with the demo script (#2561)

* [Enhance] Combined dataset supports custom sampling ratio (#2562)

* [Docs] Add MultiSourceSampler docs (#2563)

* [Doc] Refine docs (#2564)

* [Feature][MMSIG] Add UniFormer Pose Estimation to Projects folder (#2501)

* [Fix] Check the compatibility of inferencer's input/output  (#2567)

* [Fix]Fix 3d visualization (#2565)

* [Feature] Add bear example in just dance (#2568)

* [Doc] Add example and openxlab link for just dance (#2571)

* [Fix] Configs' paths of VideoPose3d (#2572)

* [Docs] update docs (#2573)

* [Fix] Fix new config bug in train.py (#2575)

* [Fix] Configs' of MotionBERT (#2574)

* [Enhance] Normalization option in 3d human pose demo and inferencer (#2576)

* [Fix] Fix the incorrect labels for training vis_head with combined datasets (#2550)

* [Enhance] Enhance 3dpose demo and docs (#2578)

* [Docs] Enhance Codecs documents (#2580)

* [Feature] Add DWPose distilled WholeBody RTMPose models (#2581)

* [Docs] Add deployment docs (#2582)

* [Fix] Refine 3dpose (#2583)

* [Fix] Fix config typo in rtmpose-x (#2585)

* [Fix] Fix 3d inferencer (#2593)

* [Feature] Add a simple visualize api (#2596)

* [Feature][MMSIG] Support badcase analyze in test (#2584)

* [Fix] fix bug in flip_bbox with xyxy format (#2598)

* [Feature] Support ubody dataset (2d keypoints) (#2588)

* [Fix] Fix visualization bug in 3d pose (#2594)

* [Fix] Remove use-multi-frames option (#2601)

* [Enhance] Update demos (#2602)

* [Enhance] wholebody support  openpose style visualization (#2609)

* [Docs] Documentation regarding 3d pose (#2599)

* [CodeCamp2023-533] Migration Deepfashion topdown heatmap algorithms to 1.x (#2597)

* [Fix] fix badcase hook (#2616)

* [Fix] Update dataset mim downloading source to OpenXLab (#2614)

* [Docs] Update docs structure (#2617)

* [Docs] Refine Docs (#2619)

* [Fix] Fix numpy error (#2626)

* [Docs] Update error info and docs (#2624)

* [Fix] Fix inferencer argument name (#2627)

* [Fix] fix links for coco+aic hrnet (#2630)

* [Fix] fix a bug when visualize keypoint indices (#2631)

* [Docs] Update rtmpose docs (#2642)

* [Docs] update README.md (#2647)

* [Docs] Add onnx of RTMPose models (#2656)

* [Docs] Fix mmengine link (#2655)

* [Docs] Update QR code (#2653)

* [Feature] Add DWPose (#2643)

* [Refactor] Reorganize distillers (#2658)

* [CodeCamp2023-259]Document Writing: Advanced Tutorial - Custom Data Augmentation (#2605)

* [Docs] Fix installation docs(#2668)

* [Fix] Fix expired links in README (#2673)

* [Feature] Support multi-dataset evaluation (#2674)

* [Refactor] Specify labels to pack in codecs (#2659)

* [Refactor] update mapping tables (#2676)

* [Fix] fix link (#2677)

* [Enhance] Enable CocoMetric to get ann_file from MessageHub (#2678)

* [Fix] fix vitpose pretrained ckpts (#2687)

* [Refactor] Refactor YOLOX-Pose into mmpose core package (#2620)

* [Fix] Fix typo in COCOMetric(#2691)

* [Fix] Fix bug raised by changing bbox_center to input_center (#2693)

* [Feature] Surpport EDPose for inference(#2688)

* [Refactor] Internet for 3d hand pose estimation (#2632)

* [Fix] Change test batch_size of edpose to 1 (#2701)

* [Docs] Add OpenXLab Badge (#2698)

* [Doc] fix inferencer doc (#2702)

* [Docs] Refine dataset config tutorial (#2707)

* [Fix] modify yoloxpose test settings (#2706)

* [Fix] add compatibility for argument `return_datasample` (#2708)

* [Feature] Support ubody3d dataset (#2699)

* [Fix] Fix 3d inferencer (#2709)

* [Fix] Move ubody3d dataset to wholebody3d (#2712)

* [Refactor] Refactor config and dataset file structures (#2711)

* [Fix] give more clues when loading img failed (#2714)

* [Feature] Add demo script for 3d hand pose  (#2710)

* [Fix] Fix Internet demo (#2717)

* [codecamp: mmpose-315] 300W-LP data set support (#2716)

* [Fix] Fix the typo in YOLOX-Pose (#2719)

* [Feature] Add detectors trained on humanart (#2724)

* [Feature] Add RTMPose-Wholebody (#2721)

* [Doc] Fix github action badge in README (#2727)

* [Fix] Fix bug of dwpose (#2728)

* [Feature] Support hand3d inferencer (#2729)

* [Fix] Fix new config of RTMW (#2731)

* [Fix] Align visualization color of 3d demo (#2734)

* [Fix] Refine h36m data loading and add head_size to PackPoseInputs (#2735)

* [Refactor] Align test accuracy for AE (#2737)

* [Refactor] Separate evaluation mappings from KeypointConverter (#2738)

* [Fix] MotionbertLabel codec (#2739)

* [Fix] Fix mask shape (#2740)

* [Feature] Add training datasets of RTMW (#2743)

* [Doc] update RTMPose README (#2744)

* [Fix] skip warnings in demo (#2746)

* Bump 1.2 (#2748)

* add comments in dekr configs (#2751)

---------

Co-authored-by: Peng Lu <[email protected]>
Co-authored-by: Yifan Lareina WU <[email protected]>
Co-authored-by: Xin Li <[email protected]>
Co-authored-by: Indigo6 <[email protected]>
Co-authored-by: 谢昕辰 <[email protected]>
Co-authored-by: tpoisonooo <[email protected]>
Co-authored-by: zhengjie.xu <[email protected]>
Co-authored-by: Mesopotamia <[email protected]>
Co-authored-by: chaodyna <[email protected]>
Co-authored-by: lwttttt <[email protected]>
Co-authored-by: Kanji Yomoda <[email protected]>
Co-authored-by: LiuYi-Up <[email protected]>
Co-authored-by: ZhaoQiiii <[email protected]>
Co-authored-by: Yang-ChangHui <[email protected]>
Co-authored-by: Xuan Ju <[email protected]>
@Ben-Louis Ben-Louis deleted the fix-vis-prediction branch December 22, 2023 03:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants