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

AVA+Context #471

Merged
merged 26 commits into from
Dec 24, 2020
Merged

AVA+Context #471

merged 26 commits into from
Dec 24, 2020

Conversation

kennymckormick
Copy link
Member

Add an AVA trick: use both RoI feature and context feature for recognition.
About 1% improvement in Top-1

HaodongDuan added 23 commits October 16, 2020 17:35
@codecov
Copy link

codecov bot commented Dec 22, 2020

Codecov Report

Merging #471 (c3df36e) into master (4df447b) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #471      +/-   ##
==========================================
- Coverage   84.69%   84.69%   -0.01%     
==========================================
  Files         118      118              
  Lines        8344     8357      +13     
  Branches     1366     1366              
==========================================
+ Hits         7067     7078      +11     
- Misses        930      932       +2     
  Partials      347      347              
Flag Coverage Δ
unittests 84.68% <100.00%> (-0.01%) ⬇️

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

Impacted Files Coverage Δ
...maction/models/roi_extractors/single_straight3d.py 84.90% <100.00%> (-0.47%) ⬇️
mmaction/core/evaluation/accuracy.py 93.18% <0.00%> (+0.03%) ⬆️

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 4df447b...c3df36e. Read the comment docs.

@innerlee innerlee merged commit dc0c82b into open-mmlab:master Dec 24, 2020
@kennymckormick kennymckormick deleted the ava+context branch January 4, 2021 05:48
# 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.

3 participants