Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[Feature] Support flip with label mapping #591

Merged
merged 7 commits into from
Feb 4, 2021

Conversation

congee524
Copy link
Contributor

fix #590

I'm doing a simple benchmark with TSM-R50 on sthv1

@codecov
Copy link

codecov bot commented Feb 3, 2021

Codecov Report

Merging #591 (fc935ee) into master (e409809) will increase coverage by 0.05%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #591      +/-   ##
==========================================
+ Coverage   84.26%   84.31%   +0.05%     
==========================================
  Files         124      124              
  Lines        8771     8774       +3     
  Branches     1467     1468       +1     
==========================================
+ Hits         7391     7398       +7     
+ Misses       1022     1019       -3     
+ Partials      358      357       -1     
Flag Coverage Δ
unittests 84.30% <100.00%> (+0.03%) ⬆️

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

Impacted Files Coverage Δ
mmaction/datasets/pipelines/augmentations.py 94.83% <100.00%> (+0.52%) ⬆️

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 e409809...c6f04db. Read the comment docs.

@congee524 congee524 force-pushed the flip_with_label_transform branch from beee7e7 to 25d950d Compare February 3, 2021 08:10
@innerlee
Copy link
Contributor

innerlee commented Feb 3, 2021

The code looks ok. @congee524 do you want to post the ckpt?

@congee524
Copy link
Contributor Author

The code looks ok. @congee524 do you want to post the ckpt?

yeah, and I'm working on it.

@dreamerlin

This comment has been minimized.

@congee524 congee524 force-pushed the flip_with_label_transform branch from 06a7073 to c6f04db Compare February 4, 2021 09:08
@congee524 congee524 changed the title Flip with label transform [Feature] Support flip with label mapping Feb 4, 2021
@innerlee innerlee merged commit 190792c into open-mmlab:master Feb 4, 2021
@congee524 congee524 deleted the flip_with_label_transform branch February 20, 2021 01:53
@dreamerlin dreamerlin mentioned this pull request Feb 25, 2021
9 tasks
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support Flip with label transform
3 participants