-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Add SSN dataset. #37
Add SSN dataset. #37
Conversation
Codecov Report
@@ Coverage Diff @@
## master #37 +/- ##
==========================================
+ Coverage 84.21% 85.02% +0.81%
==========================================
Files 73 75 +2
Lines 4175 4588 +413
Branches 638 729 +91
==========================================
+ Hits 3516 3901 +385
- Misses 554 564 +10
- Partials 105 123 +18
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
mmaction/datasets/ssn_dataset.py
Outdated
num_frames (int): Total frames of the video. | ||
label (int): Classification of the proposal. | ||
best_iou (int): The highest IOU with the groundtruth instance. | ||
overlap_self (int): Percent of the proposal's own span contained |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does overlap_self
has other candidate names? The self
is very broad
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't know, they use 'overlap_self' in mmaction1...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we can change it to something meaningful
fe80b15
to
45cabff
Compare
No description provided.