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] Refine ActionDataSample structure #2658

Merged
merged 13 commits into from
Sep 6, 2023
Merged

[Fix] Refine ActionDataSample structure #2658

merged 13 commits into from
Sep 6, 2023

Conversation

Dai-Wenxun
Copy link
Collaborator

@Dai-Wenxun Dai-Wenxun commented Aug 29, 2023

For BC-breaking:

  • replace data_sample.pred_scores.item with data_sample.pred_score
  • replace data_sample.gt_labels.item with data_sample.gt_label
  • replace data_sample.set_gt_labels() with data_sample.set_gt_label()

@codecov
Copy link

codecov bot commented Aug 30, 2023

Codecov Report

Patch coverage is 46.84% of modified lines.

❗ Current head a7a5dd9 differs from pull request most recent head 326a8d4. Consider uploading reports for the commit 326a8d4 to get more accurate results

Files Changed Coverage
mmaction/apis/__init__.py ø
...ion/models/data_preprocessors/data_preprocessor.py 0.00%
mmaction/models/heads/rgbpose_head.py 0.00%
mmaction/models/recognizers/base.py ø
mmaction/apis/inference.py 14.70%
mmaction/evaluation/metrics/acc_metric.py 44.82%
mmaction/models/losses/hvu_loss.py 50.00%
mmaction/structures/action_data_sample.py 76.47%
...maction/apis/inferencers/actionrecog_inferencer.py 100.00%
mmaction/datasets/transforms/formatting.py 100.00%
... and 7 more

📢 Thoughts on this report? Let us know!.

@cir7 cir7 closed this Sep 6, 2023
@cir7 cir7 reopened this Sep 6, 2023
@cir7 cir7 changed the title [Fix] fix gt_labels [Fix] Refine ActionDataSample Sep 6, 2023
@cir7 cir7 changed the title [Fix] Refine ActionDataSample [Fix] Refine ActionDataSample structure Sep 6, 2023
@cir7 cir7 merged commit d18ca93 into open-mmlab:dev-1.x Sep 6, 2023
9 of 19 checks passed
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